Skip to content

Disposition the surface watch's unreviewed names at Claude Code 2.1.300 - #1131

Open
seathatflowsinourveins wants to merge 1 commit into
mainfrom
claude/cc-native-practice-surface-dispositions-2300-20261015
Open

seathatflowsinourveins wants to merge 1 commit into
mainfrom
claude/cc-native-practice-surface-dispositions-2300-20261015

Conversation

@seathatflowsinourveins

@seathatflowsinourveins seathatflowsinourveins commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

Scope

  • What this PR changes: six disposition rows that the first daily run of the newly installed upstream-surface watch listed as unreviewed at Claude Code 2.1.300. These are four new environment variables and two documentation pages whose bodies changed.
  • Base commit: 929bc3c3f3b6d1f890279ff14d3f500e2d4db592, the merge base. Corrected after the GPT full1 read; a339425c is an earlier main commit in its history.
  • Lane: lane:foundation
  • Owned paths touched: catalogs/foundation/upstream-surface-dispositions.json, manifests/evidence.json

SOTA sources

Evidence-class table

Claim Evidence class Command / receipt
The catalog is valid with the new rows local_integration python3 scripts/upstream_surface_watch.py --check-dispositions gives valid, 367 rows
The watch now lists 0 unreviewed names local_integration --dry-run replay of the 01:33Z cache gives 0 unreviewed of 5 new (6 before)
The pages' load-bearing claims hold at the new digests source_review the cached bodies, at the lines quoted in the rows

Local commands run

$ python3 -m unittest tests.test_currency_due tests.test_currency_due_notice tests.test_validate tests.test_evidence_manifest
95 OK; 22 OK; 86 OK; 5 OK
$ UPSTREAM_SURFACE_TEST_CACHE=<empty dir> python3 -m unittest tests.test_upstream_surface_watch
Ran 116 tests, OK (skipped=6)
$ python3 -m unittest tests.test_upstream_surface_watch   # with this host's cache, created by the units installed at 01:32Z
FAILED (failures=4), the four CachedArtifactTests; origin/main fails identically on this host
$ python3 scripts/component_matrix.py --check; python3 scripts/new_host_grand_list.py --check
checked; passed

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

  • The four host-cache test failures are not from this change. CachedArtifactTests read 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.
  • Not in this PR: a baseline update. The baseline stays 2.1.296 until the follow-up above.

Follow-ups

From GPT full1 at c986f4a (full1-pr1131/GPT-VERDICT-c986f4ac3.md):

Checklist

  • No GitHub Actions changed.
  • No workflows changed.
  • No secrets printed, logged or committed.
  • No paid hosting, subscription or billing surface.
  • Peer-owned files and worktrees preserved.

🤖 Generated with Claude Code

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 seathatflowsinourveins added the lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers label Oct 15, 2026
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]>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant