docs: reconcile cross-doc drift after the 6-PR session (#303–#310)#311
Merged
Conversation
Full .md consistency sweep (docs-reviewer audit, all facts re-verified against compute/config.py + filesystem before applying). 13 MUST-FIX + 6 SHOULD-FIX text reconciles — no contradictions remain across the canonical docs: - Schema 0.10.10 -> 0.10.11-phase4.6 current-state in CLAUDE.md / PHASE_STATUS.md / CONTEXT.md / WORKFLOW.md / SKILL.md (PR #303 merged; was still showing the pre-bump value + a stale "in flight" framing). - Agent count 18 -> 19 (+ 4-opus/14-sonnet -> 15-sonnet, 6 -> 7 coordination flows) in PHASE_STATUS.md subagent row + CONTEXT.md roster/mapping rows (PR #304 added expert-user-explorer; PR #307 added Flow 7). - Skill count 42/38 -> 45 in .claude/agents/README.md + .claude/skills/README.md. - Cleared stale "in flight (this PR)" blocks for PRs now merged (CLAUDE.md + AGENTS.md + PHASE_STATUS.md); moved #303-#310 into the Recently-merged lists; flipped 5 PHASE_STATUS_INFLIGHT.md headers (in flight -> merged + SHA). - METHODOLOGY.md annotate line: fixed the math (was "7+5+23"=35; the proposed "7+5+26"=38 was also wrong) to the authoritative "33 declared = 7 vetoes + 26 annotates; 5 numerical guards separate". LIVE current-state facts only; dated/PR-attributed HISTORICAL narrative left intact (append-only record). ruff check . clean (whole-repo, per the lesson). Docs-only; no code/schema change. https://claude.ai/code/session_0144kHrCYNaamMPH57b7xdM7
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…miss) Follow-up to d34ab5c — the RiskFlagsCard (PR #306) header's backticks made the earlier sed pattern miss; the other 4 session headers flipped correctly. https://claude.ai/code/session_0144kHrCYNaamMPH57b7xdM7
dackclup
added a commit
that referenced
this pull request
May 30, 2026
…#326 §Gotchas (#327) Full docs-reviewer substance pass on CLAUDE.md → 5 MUST-FIX + 4 SHOULD-FIX, all fixed. CLAUDE.md only (+ PHASE_STATUS_INFLIGHT.md lockstep entry); no code/schema/compute/frontend change. MUST-FIX: §Phase status "In flight" marked PR #312 (tasteful-motion) as "THIS PR" → merged (e602485); "Recently merged" frozen at #310 → drained #311–#326 (16 entries, SHAs verified); "Next deliverables" Issue #67 flip listed pending → DONE (PR #294, USE_SECTOR_COE=True); "Section A-H/A-J" → "A-L" (helper is A-L since PR #221). SHOULD-FIX: §Stack "Phase 3b on this PR" → "(merged)" + edgartools 5.31→5.32; §Gotchas compute/main.py line refs re-anchored (840→879 · 1965-66→2084-85 · 717→728 · 785→805 · 972→1025); +2 §Gotchas for PR #326 invariants (sidebar data-rail↔globals.css pre-paint lockstep; AreaChart re-park debounce ≥300ms). Confirmed no drift: schema 0.10.11-phase4.6 · skills 46 · agents 19 · hooks 3. docs-reviewer re-check: DOCS-CLEAN (all 16 SHAs match, 5 line refs accurate, 2 gotchas code-backed). Lockstep via PHASE_STATUS_INFLIGHT.md side-file (PR #237).
dackclup
added a commit
that referenced
this pull request
May 31, 2026
…doc drift (#331) Adds the 20th subagent `financial-engineer` (Tier 3 Specialized, opus, read-only) — the team's first generative quant-design seat, the design counterpart to methodology-scientist's validation seat. Charter Phase 4-7; hands off design -> methodology-scientist (ratify) -> test-engineer (tests) -> quantrank-reviewer (review). New README Flow 8. Also drains #311-#330 doc drift (PHASE_STATUS.md was 8 PRs behind) + locksteps subagent counts (19->20 agents, 4->5 opus, 7->8 flows, Tier 3 5->6) across all six current-state doc homes. Doc + agent-infra only — no compute / schema / scoring / valuation / frontend code change. docs-reviewer + phase-coordinator Mode B gates passed (1 stale-opus-list FAIL caught + fixed). CI green: Python lint+test, Frontend build, Vercel preview all success.
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
2 participants
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
Full cross-doc consistency sweep of all
.mdfiles after this session merged 6 PRs (#303–#310). The user asked to check every.mdfor contradictions / things that don't make sense and fix them. Adocs-revieweraudit produced the fix list; every fact was re-verified againstcompute/config.py+ the filesystem before applying (and that caught a math error in the reviewer's own proposed fix — see below).Drift fixed (13 MUST-FIX + 6 SHOULD-FIX)
Schema version
0.10.10→0.10.11-phase4.6current-state (PR #303 merged but docs still showed the pre-bump value + a stale "in flight" framing): CLAUDE.md · PHASE_STATUS.md · CONTEXT.md · WORKFLOW.md · SKILL.md.Agent roster 18 → 19,
4-opus/14-sonnet→15-sonnet,6→7coordination flows (PR #304 addedexpert-user-explorer; PR #307 added Flow 7): PHASE_STATUS.md subagent row + CONTEXT.md roster + mapping-table rows. (CONTEXT.md contradicted itself — line 52 said 18, line 168 said 19.)Skill count
42/38→45:.claude/agents/README.md+.claude/skills/README.md.Stale "in flight" cleared for now-merged PRs (CLAUDE.md + AGENTS.md + PHASE_STATUS.md blocks); #303–#310 moved into the Recently-merged lists; 5
PHASE_STATUS_INFLIGHT.mdheaders flipped(in flight)→(merged, SHA).METHODOLOGY.md annotate math — current text said
7 + 5 + 23 = 35(wrong); the reviewer's proposed7 + 5 + 26 = 38was also wrong. Corrected to the authoritative breakdown: 33 declared = 7 vetoes + 26 annotates, with the 5 numerical guards listed separately.Discipline
SCHEMA_VERSION = 0.10.11-phase4.6, 19 agent files, 4 opus / 15 sonnet, 7### Flowheaders, 45 non-phase loaded skills.ruff check .clean (whole-repo, per the lesson recorded earlier this session). Docs-only — no code / schema change.Note
Excluded a mtime-only touch on
docs/agents/domain.md(identical content to HEAD) from the commit.https://claude.ai/code/session_0144kHrCYNaamMPH57b7xdM7
Generated by Claude Code