Complete Fanout-inspired learning surfaces - #89
Merged
Conversation
Migrated spec content to GitHub tracking issues. No local openspec/ directory, no openspec CLI. See fleet-workspace AGENTS.md for the new spec-driven workflow. Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Resolves four conflicts so #89 can land. Nothing in src/ conflicted. - openspec/: the branch predates nothing here -- it deletes the whole 90-file openspec tree, and git flagged only the one file main had modified, so the other 89 deletions would have gone through silently. Restored the tree from main. Given that the fleet-wide openspec removal is exactly what destroyed the task-completion records this project is still recovering from, dropping 90 more spec files as a side effect of an old branch is not a merge artifact worth accepting. - .size-limit.json: took main's per-chunk gzipped budgets over this branch's single raised total. Main's model is strictly better, and the expanded surfaces fit it -- verified, not assumed. - PROJECT_STATUS.md and docs/knowledge/learnings.md: both sides added distinct entries; kept both. - Corrected this branch's status row, which claimed it raised a gzipped JS ceiling from 8.60 MB to 8.65 MB. Main has since replaced that gate, so the sentence was no longer true. Verified on the merge result: vitest 521 passed (82 files) tsc clean size-limit entry 27.91/40 kB, react 81.98/90 kB, icons 11.01/15 kB, runtime 722 B/2 kB, css 43.01/50 kB -- all within budget blume 0 errors Co-Authored-By: Claude Opus 5 (1M context) <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
1 participant
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.
Closes #86.
This work has been finished and pushed for some time but had no pull request, so it was invisible and slowly rotting:
mainhas already cherry-picked commits off this branch, and every day it diverges further. Opening it as-is.I did not write this branch — this PR exists to land work that was already complete. Verified rather than assumed:
Verification (run in an isolated worktree at
cfcdcec)For reference,
maincurrently sits at 481 tests. This branch adds 31.git merge-treeagainstmainreports zero conflicts. The branch is 12 ahead / 4 behind.Interaction with #79
fix/79-untouched-concepts-count-as-gaps(PR #88) is open againstmainat the same time. The two branches touch no source file in common — the only overlap is four docs/task files:docs/knowledge/learnings.mddocs/product/overview.mddocs/product/surfaces.mdopenspec/changes/2026-07-26-sweep-breadth-triage/tasks.mdsrc/lib/recommend.ts,src/pages/Progress.tsx, andsrc/pages/LearnAll.tsx— the files #79 changes — are untouched here. So the two can land in either order; expect at most trivial doc conflicts on the second one.Scope
Per the issue: a complete ordered inference pathway over public Learn Inference chapters, a searchable notation/formula reference, an inspectable knowledge-topography surface, a deterministic rotating paper programme subordinate to due recovery, local-first model-routing / RAG-readiness / inference-benchmarking decision labs, focused study sessions with timed learn–retrieve–apply–explain stages, and interruption-safe lab attempts.
Includes 6 test files and 390/768/1440px screenshot verification.
Remaining
The issue's task 10 is an owner sign-off, not an implementation step. Everything else is done.
Caveat worth stating plainly:
node_moduleswas absent in a fresh worktree, so I linked the existing install rather than runningpnpm install(network installs need your say-so). CI should be treated as the authoritative run.🤖 Generated with Claude Code