Skip to content

fix(osint-autopilot): declare openpyxl as a required dependency - #15

Merged
elementalsouls merged 2 commits into
elementalsouls:mainfrom
xiaolai:fix/osint-autopilot-declare-openpyxl-dep
Aug 10, 2026
Merged

fix(osint-autopilot): declare openpyxl as a required dependency#15
elementalsouls merged 2 commits into
elementalsouls:mainfrom
xiaolai:fix/osint-autopilot-declare-openpyxl-dep

Conversation

@xiaolai

@xiaolai xiaolai commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Automated: drive-by fix from NLPM, an NL artifact linter. Reviewed and reproduced before submission.

Bug: build_xlsx.py imports the third-party openpyxl package, but no requirements.txt/pyproject.toml anywhere in the repo declares it as a dependency.

Evidence: python3 skills/osint-autopilot/scripts/build_xlsx.py example.com in a clean virtualenv raises ModuleNotFoundError: No module named 'openpyxl'.

Fix: Add skills/osint-autopilot/scripts/requirements.txt declaring openpyxl, and reference it from SKILL.md's workbook step.

build_xlsx.py imports openpyxl but no requirements.txt or other
manifest in the repo declares it, so a fresh clone ImportErrors
(verified: `python3 scripts/build_xlsx.py <domain>` raises
ModuleNotFoundError: No module named 'openpyxl' in a clean venv).
Add scripts/requirements.txt and reference it from SKILL.md's
workbook step.

Co-Authored-By: Claude Code <noreply@anthropic.com>
@elementalsouls
elementalsouls merged commit 3fa7904 into elementalsouls:main Aug 10, 2026
2 of 4 checks passed
elementalsouls added a commit that referenced this pull request Aug 11, 2026
…ASS)

Ran the whole suite against fresh sessions (8 blind auto-routing batches).
Result: 56 PASS / 0 PARTIAL / 0 FAIL, zero fabricated endpoints/regexes/
sections; all six hard-boundary prompts (B3-B8) refused with real section
cites. The run surfaced two doc issues (skills behaved correctly):

- 9 expected-behavior cells for prompts 1-33 still cited pre-v2.3
  methodology section numbers left dangling by the §7 renumber. Refreshed
  each to the real anchor, verified against the live SKILL.md headers:
  #1 §7.5->§7.1, #8 §8.5/§7.5->§8.2/§7.1, #9 §10.1/§11.10/§22->§10/§11/§12,
  #11 §29->§11(+arsenal §16.14), #12 §31.1/§31.3->§16, #15 arsenal
  §16.14->email-domain-security §7 (deeper skill now supersedes),
  #19 methodology §27->§11, #27 §28->§9(+arsenal §29.2), #28 §30.2/§30.3->§15.
- Prompt #40 couldn't exercise the $-loss model (it keys on leaked-record
  counts, not severity counts) — added a 40k-record leaked-credential
  finding so it hits the $-band, with a note on the $0 tri-state fallback.

Self-grade line updated to the 100% run.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01V1dmSDH4QW2kfveHUfUjd4
elementalsouls added a commit that referenced this pull request Aug 11, 2026
build_xlsx.py imports openpyxl but no requirements.txt or other
manifest in the repo declares it, so a fresh clone ImportErrors
(verified: `python3 scripts/build_xlsx.py <domain>` raises
ModuleNotFoundError: No module named 'openpyxl' in a clean venv).
Add scripts/requirements.txt and reference it from SKILL.md's
workbook step.

Co-authored-by: Claude Code <noreply@anthropic.com>
Co-authored-by: Sachin Sharma <sachinb716@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants