Repository navigation
fix(tools): point the stale-map line at ix map --remap - #37
Merged
Merged
Conversation
The stale-map line said "a background refresh is running", which is not always true: Ix also reports a stale map when background refresh is off (IX_MAP_BACKGROUND_REFRESH=0), when the last refresh failed, or when it could not start. On the llm path the line also replaces Ix's own map_stale record, whose "Run ix map --remap" remedy was lost. The line now gives that remedy instead. Adds unit cases for two map_stale records, a record that is not the first line, and a 1-file map. Co-Authored-By: Claude Opus 5.5 <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
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.
When
ixreports the architecture map stale,ix-subsystemsandix-mapadded a line ending "— a background refresh is running." That isn't always true: Ix also reports a stale map when background refresh is off (IX_MAP_BACKGROUND_REFRESH=0), when the last refresh failed, or when it couldn't start. On the llm path the line also replaces Ix's owndiagnostic code=map_stalerecord, so its "Run ix map --remap" remedy was lost.The line (
mapStaleLineinruntime/map-stale.ts) now reads:_Map is stale (regions as of map rev N, M files); recent edits may not be reflected — run `ix map --remap` to refresh it now._(1 filewhen singular)_Map is stale; recent edits may not be reflected — run `ix map --remap` to refresh it now._when Ix gives noregions_as_ofUpdated the assertions in
tests/map-stale.test.ts,tests/success-fixtures.test.tsand the quote inTOOL_CONTRACT.md. Added the unit cases a review found missing: twomap_stalerecords, a record that isn't the first line, and a 1-file map (parse and render).Tests (Bun 1.4.2):
bun install --frozen-lockfile,bun run typecheckclean,bun test358 pass / 11 skip (live/real-ix suites, env-gated) / 0 fail.No version change.
🤖 Generated with Claude Code