Index phase 6: polish#88
Merged
Merged
Conversation
- Wire CompileOutput.outputFormat through CSP so IndexManager dispatches TASTY vs BETASTY explicitly instead of probing the jar contents - zincCli: keep scalac flags / entry suffix / announced format in lock-step - TastyIndexer: stdout guard becomes a Resource (restores System.out on shutdown); consolidate inline try/catch into one `safe` helper - ProjectIndex: demote per-update storage logs to debug - Inject DepIndexCache and coursier Cache from the wiring layer instead of constructing them inside IndexManager - Document the jsoniter nullValue sentinel in IndexedSymbolCodecs Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Replace the four overlapping planning documents (index-plan, index-tasks, index-progress, architecture-improvements) with a single roadmap reflecting the June 2026 design review: stabilize first, build seams now, split processes later. Still-pending items from the old docs (didCreate/didRename ops, Ref-contention measurement, source-parse fallback analysis, overload disambig) are carried over; full history stays in git. Adds ai/test-infrastructure.md (TestWorkspace / TestClient spec) as a referenced companion. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
rochala
force-pushed
the
index/phase6-polish
branch
from
f1ef2d0 to
832d5e4
Compare
rochala
changed the title
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 out the index architecture rewrite (final phase of the architecture-improvements plan, on top of phase 5 / #86).
Phase 6 — polish
CompileOutput.outputFormat(TASTY/BETASTY) wired through CSP soIndexManagerdispatches explicitly instead of probing jar contents; zincCli keeps flags/suffix/format in lock-stepTastyIndexer.stdoutGuardbecomes aResource(restoresSystem.outon shutdown); inline try/catch consolidated into onesafehelperDepIndexCacheand coursierCacheinjected from the wiring layer instead of constructed insideIndexManagernullValuesentinel inIndexedSymbolCodecsDocs
ai/roadmap.md(post-design-review roadmap: stabilize first, seams now, process split later);ai/test-infrastructure.mdadded as the TestWorkspace/TestClient specTest plan
./mill __.compile— 528 targets green./mill _.test— 598 targets green (incl. cross-producer and determinism specs)🤖 Generated with Claude Code