Skip to content

Update: pin Trust to v1.1.1 and correct canonical ownership#498

Open
0xLeif wants to merge 2 commits into
mainfrom
chore/trust-1-1-1
Open

Update: pin Trust to v1.1.1 and correct canonical ownership#498
0xLeif wants to merge 2 commits into
mainfrom
chore/trust-1-1-1

Conversation

@0xLeif

@0xLeif 0xLeif commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Pins the CorvidLabs Trust gate in .github/workflows/trust.yml from 9d32b5786d2e9e4d39fc581c0091c721ee3d4226 # v1.0.0 to a239f78658e5ad0f12fa230f494890e40c6e4d7b # v1.1.1 (immutable commit SHA).
  • Corrects canonical ownership recorded on CHG-0001-adopt-trust-1-and-specsync-5 (SpecSync 5.1.1 change correct-owner):
    • src/spec/commands.rs β†’ owned by spec module spec (per specs/spec/spec.spec.md frontmatter files:)
    • src/templates.rs β†’ owned by spec module templates (per specs/templates/templates.spec.md frontmatter files:)
  • Re-approves, re-verifies, and accepts CHG-0001 with the note: "Re-acceptance with canonical ownership correction for src/spec/commands.rs; canonical deltas were already integrated and were not replayed."

Background

specsync change accept CHG-0001-adopt-trust-1-and-specsync-5 was blocked by:

error: acceptance input 'src/spec/commands.rs' is production source without deterministic canonical ownership

(and subsequently the same for src/templates.rs). CHG-0001 is a no-spec change (affected_specs empty), so the sanctioned SpecSync 5.1.1 per-path fix specsync change correct-owner was applied against the canonical spec modules whose frontmatter files: lists already include these paths. No spec files were edited; the only non-.specsync/ change is the Trust pin in .github/workflows/trust.yml.

Verification

  • specsync change verify CHG-0001-adopt-trust-1-and-specsync-5 β€” passed (verify-native lane: 962 unit tests + all integration suites green, build, template validation; 10 templates, 0 errors).
  • Final specsync check β€” clean (exit 0):
βœ“ SDD lifecycle valid (2 active change(s))
All specs unchanged β€” nothing to validate.
File coverage: 65/65 (100%)
LOC coverage:  33772/33772 (100%)

@0xLeif
0xLeif requested a review from a team as a code owner July 18, 2026 22:37
@0xLeif
0xLeif requested review from 0xGaspar, Kyntrin and tofu-ux July 18, 2026 22:37
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❌ Corvin says...

      _
    <(;\  .oO(oh no...)
     |/(\
      \(\\
      " "\\

"I'm pecking through the errors..."

CI Summary

Check Status
Dependency Audit βœ… Passed
Integration (3 OS) ❌ cancelled
Lint (fmt + clippy) βœ… Passed
Spec Validation ❌ failure
Tests (3 OS) βœ… Passed

Powered by corvid-pet

@github-actions
github-actions Bot dismissed their stale review July 18, 2026 23:18

Superseded by updated review.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

βœ… Corvin says...

      _
    <(^\  .oO(Caw! ^v^)
     |/(\
      \(\\
      " "\\

"Looking sharp! Like a beak should be."

CI Summary

Check Status
Dependency Audit βœ… Passed
Integration (3 OS) βœ… Passed
Lint (fmt + clippy) βœ… Passed
Spec Validation βœ… Passed
Tests (3 OS) βœ… Passed

Powered by corvid-pet

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