Skip to content

Three submodule registrations were deleted by a commit about a skill document - #2755

Merged
gHashTag merged 2 commits into
masterfrom
w699-submodules
Aug 28, 2026
Merged

Three submodule registrations were deleted by a commit about a skill document#2755
gHashTag merged 2 commits into
masterfrom
w699-submodules

Conversation

@gHashTag

Copy link
Copy Markdown
Owner

Refs #2754

tri rtl check chips/phi in cli-tri.yml has been failing with

Error: No such file or directory (os error 2)

which names neither the path nor what was expected there. The gate then reports tri rtl check emitted 0 verdicts. Two defects sat behind that one line.

The directory is not there

.gitmodules declares chips/{phi,euler,gamma}, and the repository had zero gitlink entries — so submodules: true in the workflow, already set by someone who had got this far before, fetched nothing at all.

They were registered on 2026-05-23 in 1bfd31bcf and deleted in b79702ee1, a 174-file commit whose subject is about a CI-gates skill document. Three deletions, buried:

chips/euler | 1 -
chips/gamma | 1 -
chips/phi   | 1 -

Restored to the exact SHAs they held before that commit, each verified to still exist upstream via the API:

path commit
chips/euler 73b9f0a050902300a3c29208ac2d493ebc642213
chips/gamma a90a3d04bc50943949b1e77469069b55f4bc6118
chips/phi f5456685c3593665153fe2765c85bb1f46ec14c2

Not a new decision — the same three lines that were removed, put back.

The tool did not say so

declared_sources names the file it cannot read. top_from_info runs first and did not, so the whole command surfaced as a bare OS error. It now refuses with the path and the expectation:

Error: no design directory at chips/phi -- `tri rtl check` expects a directory
holding info.yaml and src/. If this is a submodule path, it is declared but not
checked out.

That message is what would have found the missing gitlinks in a minute rather than a session.

🤖 Generated with Claude Code

gHashTag and others added 2 commits August 28, 2026 22:41
…the directory that is missing

`tri rtl check chips/phi` in cli-tri.yml has been failing with

    Error: No such file or directory (os error 2)

which names neither the path nor what was expected there, and the gate then
reports "emitted 0 verdicts". Two separate defects sat behind it.

1. THE DIRECTORY IS NOT THERE. .gitmodules declares chips/{phi,euler,gamma},
   and the repository had ZERO gitlink entries, so `submodules: true` in the
   workflow -- already set, by someone who had got this far before -- fetched
   nothing.

   They were registered on 2026-05-23 in 1bfd31b and deleted in b79702e,
   a 174-file commit whose subject is about a CI-gates SKILL DOCUMENT. Three
   deletions, buried. Restored to the exact SHAs they held before that commit,
   each verified to still exist upstream:

       chips/euler  73b9f0a050902300a3c29208ac2d493ebc642213
       chips/gamma  a90a3d04bc50943949b1e77469069b55f4bc6118
       chips/phi    f5456685c3593665153fe2765c85bb1f46ec14c2

2. THE TOOL DID NOT SAY SO. `declared_sources` names the file it cannot read;
   `top_from_info` runs FIRST and did not, so the whole command came out as a
   bare os error. It now refuses with the path and the expectation, and says
   that a submodule path may be declared without being checked out.

Refs #2754

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
…path

Refs #2754

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

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-08-28 15:42:40 UTC

Summary

Status Count
Total Open PRs 8
PRs with Failing Checks 7
PRs with All Checks Green 1
READY 0
FAILING 7
PENDING 0

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=25a9c40dc273 != 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).

@github-actions

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@gHashTag
gHashTag merged commit 4ad209e into master Aug 28, 2026
31 of 33 checks passed
@gHashTag
gHashTag deleted the w699-submodules branch August 28, 2026 15:56
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.

1 participant