Repository navigation
Rename and rewrite all skills; fix 13D scan - #8
Open
eggmasonvalue wants to merge 2 commits into
Open
eggmasonvalue wants to merge 2 commits into
eggmasonvalue wants to merge 2 commits into
Conversation
…gotchas - Rename signal-sweep -> idea-generation, sec-edgar-skill -> sec-filings, market-scout -> market-data, bottom-up-analyst -> due-diligence; signal-sweep-cache -> idea-cache (IDEA_CACHE_DIR) - Data skills: job -> command plus hard-won gotchas from the scripts and live checks (filing-date selectors, 10-Q Part namespacing, 13D form rename, yfinance unit and share-class quirks, lagging screener fields) - due-diligence: drop textbook material; archetypes in one file; ownership semantics single-sourced in sec-filings - Fix scan_insiders 13D scan: SEC renamed forms to SCHEDULE 13D (old names returned nothing); dedupe by accession; issuer from schedule - fetch_market_data: distinguish class shares from implied shares - Remove list_headings.py (rg -n '^#')
SKILL.md now holds routes, commands, and pointers only. Flag semantics moved to --help, recoverable mistakes to error hints, report caveats stay in report text, and off-script quirks to references (yfinance.md, hunting.md, edgartools.md, ownership.md).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this changes
Renames.
signal-sweep→idea-generation,sec-edgar-skill→sec-filings,market-scout→market-data,bottom-up-analyst→due-diligence.pitch-like-loukeeps its name. The cachesignal-sweep-cache→idea-cache(envIDEA_CACHE_DIR). References are updated in package.json, bootstrap.mjs, the insider-scan workflow, pyproject, .gitignore, markdownlint config, AGENTS.md, context/, and the READMEs. In the skill lockfiles only the project-name line changed (edited by hand to avoid a uv format bump).Where documentation lives (second commit; the first wrongly put gotchas in SKILL.md):
--help: flag semantics (filing-date selectors, 10-Q Part naming, 20-F items,ex-99.1).BRK.B→BRK-B).references/: off-script detail. New files:sec-filings/references/{edgartools,ownership}.md,market-data/references/yfinance.md,idea-generation/references/hunting.md.due-diligence rewrite. Textbook material is cut. Six archetype files are merged into
archetypes.md(edge, traps, valuation route, tell). The guides becamenormalization.md,valuation.md, andmemo.md.guide_competitive.mdis folded into SKILL.md. Ownership interpretation now lives only insec-filings/references/ownership.md. Scope is unchanged: long-only, and banks, insurers, REITs, funds, and binary assets stay excluded.pitch-like-lou. Tightened. The description now also triggers on a "stock pitch" or a "punchier rewrite".
Script fixes
scan_insiders.py: the 13D scan returned nothing because EDGAR now usesSCHEDULE 13D/13D/A; queryingSC 13Don 2026-09-15 returned 0 rows. The scan now queries both names, dedupes by accession (the index lists each schedule under both filer and subject), and takes the issuer fromissuer_info(the old code could attribute a stake to the filer, e.g. FTK → ProFrac). Live test on 2026-09-15: 17 schedules.fetch_market_data.py: shows class shares and implied all-class shares separately when they differ. Prints a hyphen hint for dotted tickers.fetch_transcripts.py: adds the hyphen hint and a vendor-transcript note in each file.scan_market.py: reports for screens usinglastclose52week*fields now carry a lag warning.fetch_filing.py,parse_financials.py,fetch_filings.py:--helptext only.list_headings.py(it wasrg -n '^#').context/DECISIONS.md: the z-score entry now says 22 trading days vs the prior 252 rolling returns, not "30-day".Known open issues (not fixed here)
lastclose52weeklow/high.lasttwelvemonthsscreener fields lag the live quote. On 2026-10-31, a≤ 1.0query returned stocks 6–50% above their low. Sonear-52wk-lowandfallen-from-gracegive misleading results until they are redesigned. This PR only adds a warning.dcf.pyandepv.pyneed Python 3.10+ (they crash on 3.9 outside the secpi venv).fetch_transcripts.py --latesttrusts Yahoo's listing order. The Q&A split depends on an operator turn containing "question".scan_insiders.pyclusters fall back to name matching when an owner CIK is missing.Verification
ruff check,ruff format --check, andmarkdownlint-cli2pass.^TNX), screener lag, the BRK.B hint, and the screen-report warning.fetch_filing.py/parse_financials.pyafter the--helpedits (only--helpwas exercised), transcripts, theme search, and the conference scan.These checks used the first-commit docs, before SKILL.md was slimmed, and were not re-run.