Repository navigation
Disposition the surface watch's unreviewed names at Claude Code 2.1.300 - #1131
Open
seathatflowsinourveins wants to merge 1 commit into
Open
seathatflowsinourveins wants to merge 1 commit into
seathatflowsinourveins wants to merge 1 commit into
Conversation
The first daily run of the installed surface watch (command center, W1, 2026-10-15T01:32Z) listed six unreviewed names against the 2.1.296 baseline. - claude:env:CLAUDE_CODE_ARTIFACT_PUBLISH_EDITS (2.1.300): not-applicable; the Artifact tool is off here (enableArtifact false). - claude:env:CLAUDE_CODE_DISABLE_FILE_MENTIONS (2.1.299): declined; a context saving changes a default only with paired quality parity. - claude:env:CLAUDE_CODE_IDLE_COMPACT_MIN_TOKENS (2.1.299): default-on; idle compaction was screened and adopted at its 200K default. - claude:env:CLAUDE_CODE_YOU_SHOULD_KNOW_DEBUG (2.1.287): declined; a debugging aid only. - claude:doc:memory and claude:doc:skills: re-read at their new digests (1ebf1bdc..., 4d090296...). The claims the rows rest on still hold: @path imports, the AGENTS.md load table and auto memory's first 200 lines or 25KB; skills re-attached after compaction at 5,000 tokens each and 25,000 combined. The digests move to the new bodies. Each env row cites https://code.claude.com/docs/en/env-vars. The rows are spliced as text at indent=1, as the catalog requires. Checks: --check-dispositions valid (367 rows); the replayed report shows 0 unreviewed of 5 new. test_currency_due 95 OK, test_currency_due_notice 22 OK, test_validate 86 OK. test_upstream_surface_watch with no host cache: 116 OK (6 skipped). With this host's new cache (created by W1), four CachedArtifactTests fail on pinned counts, and they fail identically at origin/main. Co-Authored-By: Claude Opus 5.5 <[email protected]>
seathatflowsinourveins
added a commit
that referenced
this pull request
Oct 15, 2026
…nvariant The command center's FULL read of #1155 at bf9153a (PASS, one fold): the four env names the 2.1.300 re-baseline added (CLAUDE_CODE_ARTIFACT_PUBLISH_EDITS, CLAUDE_CODE_DISABLE_FILE_MENTIONS, CLAUDE_CODE_IDLE_COMPACT_MIN_TOKENS, CLAUDE_CODE_YOU_SHOULD_KNOW_DEBUG) had no disposition row, so the baseline grandfathered them out of unreviewed and currency_due (upstream_surface_watch.py:1597-1600). Each gets a baseline-unreviewed row (the catalog's value for this case, upstream_surface_watch.py:210-220, docs/upstream-surface-watch.md:192-193); #1131's rebase turns them into rulings. test_every_name_a_re_baseline_added_has_a_disposition_row compares the baseline with the 2.1.296 baseline at f165498 (from git history) and requires a row for every added name. Names that predate that baseline (1,566) are grandfathered under the earlier rule, so the invariant covers re-baseline additions, not every baseline name. Fail-before at bf9153a: the four names. After: tests.test_upstream_surface_watch, test_currency_due and test_currency_due_notice OK; --check-dispositions valid (367 rows); validate rc 0. Co-Authored-By: Claude Opus 5.5 <[email protected]>
5 tasks done
This branch has not been deployed
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.
Scope
929bc3c3f3b6d1f890279ff14d3f500e2d4db592, the merge base. Corrected after the GPT full1 read;a339425cis an earlier main commit in its history.lane:foundationcatalogs/foundation/upstream-surface-dispositions.json,manifests/evidence.jsonSOTA sources
54351937…). Rows forCLAUDE_CODE_ARTIFACT_PUBLISH_EDITS(v2.1.300),CLAUDE_CODE_DISABLE_FILE_MENTIONS(v2.1.299),CLAUDE_CODE_IDLE_COMPACT_MIN_TOKENS(v2.1.290+, default 200000, floor 100000) andCLAUDE_CODE_YOU_SHOULD_KNOW_DEBUG(v2.1.287).94506ff0d3bd9185f1752da36e968775021fb48f, sections 2.1.299 and 2.1.300.1ebf1bdc…and4d090296…). The memory page still documents@pathimports, the AGENTS.md load table and auto memory's first 200 lines or 25KB. The skills page still re-attaches each invoked skill after compaction at its first 5,000 tokens, within a combined 25,000.Evidence-class table
local_integrationpython3 scripts/upstream_surface_watch.py --check-dispositionsgives valid, 367 rowslocal_integration--dry-runreplay of the 01:33Z cache gives 0 unreviewed of 5 new (6 before)source_reviewLocal commands run
Decision record
No new decision. The rows follow the dispositions catalog's own rules and the surface-watch guide (docs/upstream-surface-watch.md).
Notes for the reads
CachedArtifactTestsread optional real vendor artifacts from the host's state directory. They skip when it is absent, as in CI and on this host until the daily units were installed tonight. They pin counts from an older vintage of those artifacts: 171 and 173 settings, 1034 Codex config keys. Today's bodies have 172, 174 and 1057. The follow-up is to re-baseline the watch to 2.1.300 and tie those counts to the baseline. That is a separate change for the command center to rule on.Follow-ups
From GPT full1 at c986f4a (
full1-pr1131/GPT-VERDICT-c986f4ac3.md):tests/test_upstream_surface_watch.py:879,:892,:903,:939). Re-baseline the upstream surface watch at Claude Code 2.1.300; hold cached-artifact checks to the baseline #1155's re-baseline closes them, so Re-baseline the upstream surface watch at Claude Code 2.1.300; hold cached-artifact checks to the baseline #1155 lands first and this PR rebases onto it.Checklist
🤖 Generated with Claude Code