Skip to content

Port gHashTag/trios:crates/trios-chat/rings/CR-CHAT-05/src/store_compaction_integrity_guard.rs (Rust, 1 function) to spe - #7920

Merged
gHashTag merged 3 commits into
masterfrom
queen-7895
Oct 12, 2026
Merged

gHashTag merged 3 commits into
masterfrom
queen-7895

Conversation

@gHashTag

@gHashTag gHashTag commented Oct 8, 2026

Copy link
Copy Markdown
Owner

Closes #7895

Written by a bee on queen-7895 and published by tools/queen/publish.py. The branch itself is the bee's; the second commit is the coordination entry every pull request must add, which a bee has no way to know about.

1 file changed, 154 insertions(+)

🤖 Generated with Claude Code

@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-08 20:11:00 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 46
PRs with All Checks Green 4
READY 1
FAILING 46
PENDING 0
NO CHECKS YET 0

These columns do not partition: 1 + 46 + 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)=3c0ade9e73e4 != 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).

This was referenced Oct 8, 2026
This was referenced Oct 10, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-11 18:58:27 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 39
PRs with All Checks Green 11
READY 0
FAILING 39
PENDING 0
NO CHECKS YET 0

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

Seal Status

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

Trinity Bee and others added 2 commits October 12, 2026 14:29
…action_integrity_guard.rs to .t27

- Add CompactionIntegrityError enum with all 6 variants
- Port validate_compaction function with all 6 validation rules
- Add all 10 tests from original (SCPI-01..SCPI-10)
- All tests pass, 100% success rate

Closes #7895
A pull request must add exactly one docs/now entry and a bee has no way
to know that: its brief names a boundary file and acceptance criteria,
and docs/now/ is neither. The publisher adds it rather than failing the
gate.

Closes #7895

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

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-12 07:38:48 UTC

Summary

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

Seal Status

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

…7920)

The issue's boundary names the spec file alone ('Write this one file and
nothing else').

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

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-12 07:40:09 UTC

Summary

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

Seal Status

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

@tri-claw tri-claw Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Reviewer bee verdict for head d25aaf5373b5563a560b34c6458e958d896600ae (tools/bees/reviewer.py, zai glm-4.7-flash, 3 turns, 207 s; then glm-4.5-flash, 4 turns, 96 s).

BEE-VERDICT: APPROVE
summary: Port of validate_compaction to .t27 with all 10 SCPI tests passing
criterion: File exists and parses -- met -- evidence: brief.md:761-767
discounted-check: Documented t27c subcommands exist -- failure exists on master e6eb329
discounted-check: cli-tri -- failure exists on master e6eb329
discounted-check: t27b-native-linux -- failure exists on master e6eb329
discounted-check: t27b-native-macos -- failure exists on master e6eb329
discounted-check: t27b-native-ratchet -- failure exists on master e6eb329

Evidence

Evidence

1. Does the change do what issue #7895 asks?

The issue requires porting crates/trios-chat/rings/CR-CHAT-05/src/store_compaction_integrity_guard.rs to .t27 format with the validate_compaction function.

All acceptance criteria are met:

  • Criterion 1: test -f specs/port/trios/crates/trios-chat/rings/CR-CHAT-05/src/store_compaction_integrity_guard.t27 && echo present prints present -- Runner result: present (line 761)

  • Criterion 2: grep -cE '^\s*(pub )?fn (validate_compaction)\(' specs/port/trios/crates/trios-chat/rings/CR-CHAT-05/src/store_compaction_integrity_guard.t27 prints 1 -- Runner result: 1 (line 762)

  • Criterion 3: t27c gen specs/port/trios/crates/trios-chat/rings/CR-CHAT-05/src/store_compaction_integrity_guard.t27 > /tmp/t27-gen.zig && grep -c 'not yet implemented' /tmp/t27-gen.zig prints 0 and t27c gen ... | wc -l prints at least 13 -- Runner results: 0 and 146 (lines 763-764)

  • Criterion 4: t27c spec-status specs/port/trios/crates/trios-chat/rings/CR-CHAT-05/src/store_compaction_integrity_guard.t27 does not print NOPARSE -- Runner result: IMPLEMENTED (line 765)

  • Criterion 5: grep -cE '^[[:space:]]*test[[:space:]]+("|[A-Za-z_])' specs/port/trios/crates/trios-chat/rings/CR-CHAT-05/src/store_compaction_integrity_guard.t27 prints at least 1 -- Runner result: 10 (line 766)

  • Criterion 6: t27c test-report specs/port/trios/crates/trios-chat/rings/CR-CHAT-05/src/store_compaction_integrity_guard.t27 2>&1 | grep -c BLOCKED prints 0 -- Runner result: 0 (line 767)

The port correctly implements all 10 tests from the original Rust code (SCPI-01 through SCPI-10), validating:

  • Counter gaps are rejected (scpi_01)
  • Count mismatches are rejected (scpi_02)
  • Zero anchors are rejected (scpi_03)
  • Non-monotonic counters are rejected (scpi_04)
  • Empty remaining chains are rejected (scpi_05)
  • Removed counts exceeding original are rejected (scpi_06)
  • Valid compactions are accepted (scpi_07)
  • No removal scenarios work (scpi_08)
  • Single remaining cases work (scpi_09)
  • Empty original states work (scpi_10)

No edits outside the boundary: Only the target file was added.

No hand edits under gen/: The diff shows only the new .t27 file.

No deleted tests: All 10 original tests are present.

No weakened assertions: All tests make assertions matching the original Rust behavior.

No non-ASCII identifiers: All identifiers are ASCII.

No secrets: No credentials or sensitive data present.

The spec matches its tests: The function validates exactly what the tests check.

2. For EVERY red check listed in brief.md: does it count against THIS head?

Check On master? Why it doesn't count against this head
Documented t27c subcommands exist YES - failure on master e6eb329 The failure exists on master for the same reason; not introduced by this PR
cli-tri YES - failure on master e6eb329 The failure exists on master for the same reason; not introduced by this PR
t27b-native-linux YES - failure on master e6eb329 The failure exists on master for the same reason; not introduced by this PR
t27b-native-macos YES - failure on master e6eb329 The failure exists on master for the same reason; not introduced by this PR
t27b-native-ratchet YES - failure on master e6eb329 The failure exists on master for the same reason; not introduced by this PR

All red checks are pre-existing failures on master, not defects introduced by this pull request.

Second, independent review (glm-4.5-flash): APPROVE -- Successfully ported validate_compaction function to .t27 with all 10 original test cases

@tri-claw tri-claw Bot added the bee-reviewed A reviewer bee reviewed and verified this PR at its current head; the only merge signal (#5525) label Oct 12, 2026
@gHashTag
gHashTag enabled auto-merge (squash) October 12, 2026 08:17
@gHashTag
gHashTag merged commit 6a45537 into master Oct 12, 2026
27 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bee-reviewed A reviewer bee reviewed and verified this PR at its current head; the only merge signal (#5525)

Projects

None yet

2 participants