Skip to content

test(capture): remove obsolete unbound owner deletion expectation - #1490

Merged
drewstone merged 3 commits into
mainfrom
fix/remove-obsolete-owner-capture-mock-20261001
Oct 1, 2026
Merged

drewstone merged 3 commits into
mainfrom
fix/remove-obsolete-owner-capture-mock-20261001

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

Change

Remove the obsolete positive mock row that expects owner deletion without a dispatched native control reference.
Its logical ID also differs from the recorded owner input ID.
Remove the dead branches attached to that row.
Preserve all four negative refusal controls.
Execution source and validation are unchanged.

Actual proof

The immutable managed consumer run i already proved deletion with both exact identities:
all 3,766 native files verified, root and child automatically absent, checkpoint cleanup confirmed.
Failed f/g and preserved h evidence remain retained.
See Lab PR1209 and Runtime1487 independent review.

Publication

Runtime0.290.0 publication stopped on this single obsolete expectation.
The failed tag remains immutable.
A new patch release will publish through the maintained workflow after this repair merges.
No new tests or local suites.

tangletools
tangletools previously approved these changes Oct 1, 2026

@tangletools tangletools 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.

✅ Auto-approved PR — 09429460

Blanket team auto-approval is intentional. The merge gates are CI and codex-p1.
No automated review runs on this PR. This approval rests on the rule above alone.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-10-01T12:34:34Z

tangletools
tangletools previously approved these changes Oct 1, 2026

@tangletools tangletools 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.

✅ Auto-approved PR — 641b70a1

Blanket team auto-approval is intentional. The merge gates are CI and codex-p1.
No automated review runs on this PR. This approval rests on the rule above alone.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-10-01T12:35:34Z

@tangletools tangletools 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.

✅ Auto-approved PR — 96fb6a29

Blanket team auto-approval is intentional. The merge gates are CI and codex-p1.
No automated review runs on this PR. This approval rests on the rule above alone.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-10-01T12:36:34Z

@drewstone

Copy link
Copy Markdown
Contributor Author

Independent review passed for96fb6a299504604ee559574621e73374007b54be. Removes only the obsolete false-positive row that lacks an accepted native binding and uses a mismatched logical ID. Four negative preservation cases remain. Exact source validation is untouched. Real maintained Sandbox execution i already verified complete capture and automatic root/child cleanup; f/g/h retain predecessor refusals. The patch note accurately describes this release-verification correction. No new tests or weakened cleanup authority.

@drewstone
drewstone merged commit 99af305 into main Oct 1, 2026
4 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.

2 participants