docs(workflow-md): archive Phase 0-3 (v1.0) → docs/archived/ — 1732 → 1460 lines (Optimization PR D)#144
Merged
Merged
Conversation
Fourth PR in the .md optimization sequence (Option D). WORKFLOW.md was 1732 lines because it accumulated complete task lists + acceptance criteria for every shipped phase. Phase 0-3 (v1.0) shipped 2026-05-14 and is historical now — its content is post-mortem documentation, not forward-looking guidance. WORKFLOW.md (1732 → 1460 lines, -16%): - L196-L473 (Phase 0 / 1 / 2 / 3 + PR 3c / 3d / 3e detail + Phase 3 acceptance) extracted verbatim into a new archive file - Replaced with a 7-line pointer: archive location + v1.0 ship date + "current work starts at Phase 4 below" New file docs/archived/PHASE_0_3_WORKFLOW.md (290 lines): - Header notes archive date + v1.0 ship date + back-link to WORKFLOW.md as the live forward-looking source - Body preserves the original Phase 0-3 content verbatim — no edits to historical record CLAUDE.md (180 → 181 lines, per-PR lockstep): - §Phase status "Recently merged" — added PR #143 (B sync + dedup); dropped #124 (now 2 entries past the bar) - §Phase status — replaced "PR C in flight" tracker with "PR D in flight" note covering archive scope AGENTS.md (343 → 343 lines, per-PR lockstep): - §Phase + version state — optimization sequence tracker updated: PR C ✅ → PR D in flight, PR E-G remaining Why this matters for Claude effectiveness: - WORKFLOW.md is referenced on-demand (not auto-loaded), but when Claude reads it for "what's the current Phase X plan?", it previously waded through 273 lines of completed work first. Now Phase 4 starts at L205 instead of L478 → faster orientation. - Archive preserves the v1.0 build history for anyone who wants to understand how the project bootstrapped — Phase 8 (S&P 1500 universe expansion) PRs will likely reference Phase 0-3 patterns. What this PR does NOT touch: - Code · schemas · CI workflows · pre-merge-prod-sim.yml won't trigger Next in sequence: PR E (SKILL.md restructure + TOC) · PR F (skill description audit ×38) · PR G (PHASE_STATUS.md "Current State" summary at top).
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
dackclup
added a commit
that referenced
this pull request
May 20, 2026
… PR E) (#145) Fifth PR in the .md optimization sequence (Option D). SKILL.md is the long-form rulebook (Rules 1-18 + schema-version table + library matrix). It's referenced on-demand, so size matters less than for CLAUDE.md — but the heavy duplication with CLAUDE.md (§Stack) and AGENTS.md (§Project structure) bloated it for no marginal value. SKILL.md (641 → 574 lines, -10%): Added: - §Contents — table-of-contents with anchor links to every section - §"Rules at a glance" — 1-row-per-rule summary table for Rules 1-18, with "Most-cited from" column so readers can find the source of truth fast (e.g., Rule 16 → CLAUDE.md §Conventions + top5-rotation-audit skill; Rule 18 → CLAUDE.md + WORKFLOW.md + portable-observability-before-wiring skill) Dedup'd to reference canonical sources: - §Required Tech Stack (68 → 39 lines): full Python-libraries block dropped (was duplicating CLAUDE.md §Stack). Kept the phase-specific stretch additions (Kaggle / Modal / Whisper / factor libs) + the optional-dep license table by phase. Added mlfinlab-is-BANNED note (was buried earlier). - §Mandatory Repository Structure (97 → 26 lines): 90-line ASCII file tree duplicated AGENTS.md §Project structure with stale annotations. Replaced with a module-level table that locks the per-phase responsibility breakdown (ingest / features / scoring / valuation / ml / portfolio / output) — the structural contract future phases must respect. Rules 1-18 kept verbatim. Rule numbers are stable inbound-reference anchors — renumbering would break CLAUDE.md / AGENTS.md citations. CLAUDE.md (181 → 181, per-PR lockstep): - §Phase status "Recently merged" — added PR #144 (D archive); dropped #140 (now 4 entries past the bar) - §Phase status — replaced "PR D in flight" tracker with "PR E in flight" note covering SKILL.md restructure scope AGENTS.md (343 → 343, per-PR lockstep): - §Phase + version state — optimization sequence tracker updated: PR D ✅ → PR E in flight, PR F-G remaining Why this matters for Claude effectiveness: - The Rules-at-a-glance table makes "Rule 16" / "Rule 18" lookups one scroll instead of a search. When CLAUDE.md says "see Rule 18", Claude can now find which other docs cite that rule (so a change to Rule 18 wording / interpretation can be propagated correctly). - Dedup'd Tech Stack + Repo Structure sections were dead weight when Claude already has those facts from CLAUDE.md / AGENTS.md. Now SKILL.md adds only what THOSE files don't have (phase-stretch deps with license posture; module-level phase ownership). Next in sequence: PR F (skill description audit ×38 — TRIGGER / SKIP clarity + length normalization) · PR G (PHASE_STATUS.md "Current State" summary at top). Co-authored-by: Claude <noreply@anthropic.com>
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.
Summary
PR D in the
.mdoptimization sequence (Option D). WORKFLOW.md was 1732 lines because it accumulated complete task lists + acceptance criteria for every shipped phase. Phase 0-3 (v1.0) shipped 2026-05-14 — its content is post-mortem documentation, not forward-looking guidance.This PR moves the v1.0 build history to an archive doc and leaves WORKFLOW.md with only active / upcoming phase work.
Numbers
docs/archived/PHASE_0_3_WORKFLOW.mdNet WORKFLOW.md saved 272 lines. Forward-looking orientation: Phase 4 now starts at L205 (was L478 — Claude reading "what's the current Phase X plan?" no longer wades through completed work first).
What's archived
All preserved verbatim in
docs/archived/PHASE_0_3_WORKFLOW.mdwith a header noting:v1.0.0-phase3eWORKFLOW.mdas the live forward-looking sourceReplacement in WORKFLOW.md
Where Phase 0-3 used to live, 7 lines:
Why this matters for Claude effectiveness
WORKFLOW.md is referenced on-demand, not auto-loaded — but every time Claude reads it for "what's the current Phase X plan?" or "what's the next sub-PR?", it previously had to skip past 273 lines of completed work. Faster orientation = better task-relevant context.
The archive preserves the v1.0 build history. Future Phase 8 work (S&P 1500 universe expansion) will likely reference Phase 0-3 patterns when adding similar bootstrap-level scaffolding for new constituents.
What this PR does NOT touch
compute/,frontend/,tests/)pre-merge-prod-sim.ymlwon't trigger (path filter excludes.md)Next in sequence (NOT this PR)
Test plan
pre-merge-prod-sim.ymldoes not triggerdocs/archived/PHASE_0_3_WORKFLOW.mdback-link resolveshttps://claude.ai/code/session_015649aRyi2bvciQYZVNACd2
Generated by Claude Code