Skip to content

ledger.t27: neutral checkout paths in the sweep tests, so the devhome scan is green again (Closes #10653) - #10654

Merged
gHashTag merged 1 commit into
masterfrom
t27o/devhome-paths-fix
Oct 18, 2026
Merged

gHashTag merged 1 commit into
masterfrom
t27o/devhome-paths-fix

Conversation

@gHashTag

Copy link
Copy Markdown
Owner

Closes #10653. Part of #10522 (repairs a defect of my own lane X, merged with #10646).

specs/tools/t27o/ledger.t27 had 9 /Users/playom/... literals in the cache_path / repo_is_here tests; tools/check_devhome_paths.py (the scan check) reported FAIL new specs/tools/t27o/ledger.t27 (9 occurrence(s)), so scan is red on every PR after #10646. The rule under test reads only the last path component, so the literals become /srv/checkouts/... with every expected value unchanged.

  • t27b test specs/tools/t27o/ledger.t27: 29/29
  • python3 -I tools/check_devhome_paths.py: OK: 29 file(s), 37 occurrence(s), all pinned in tools/devhome_baseline.txt
  • t27c gen-rust of the spec: byte-identical to the committed gen/rust/tools/t27o/ledger.rs (tests are not lowered), so no gen change
  • seal re-saved, --verify MATCH

🤖 Generated with Claude Code

… scan is green again (Closes #10653)

The cache_path and repo_is_here tests used /Users/playom/... literals
(9, merged with #10646); tools/check_devhome_paths.py reported them as
new and turned the scan red for every later pull request. The rule reads
only the last path component, so /srv/checkouts/... keeps every expected
value. t27b 29/29; the devhome check prints OK; gen-rust unchanged
(tests are not lowered); seal re-saved, --verify MATCH.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-18 21:21:34 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 45
PRs with All Checks Green 5
READY 2
FAILING 45
PENDING 0
NO CHECKS YET 0

These columns do not partition: 2 + 45 + 0 + 0 = 47, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=fad12492937a != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@gHashTag
gHashTag merged commit 382a5ba into master Oct 18, 2026
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] scan red on master: ledger.t27 tests carry 9 developer home paths (from #10646)

2 participants