Skip to content

📚 docs: backlog: record COR-104-T10 delivery - #487

Merged
pascalandy merged 3 commits into
mainfrom
cor-104-t10-done-1
Nov 4, 2026
Merged

pascalandy merged 3 commits into
mainfrom
cor-104-t10-done-1

Conversation

@pascalandy

@pascalandy pascalandy commented Nov 4, 2026 •

Copy link
Copy Markdown
Owner

Video-convert completed the verification loop, but Nicely still lists COR-104-T10 as pending. This records PR #17, its v0.2.2 tag, and the gate evidence.

COR-104 is Shipped, and just next now selects COR-102-T1. Pascal's UAT and gate-removal decisions remain pending under After. The linked adoption details name the five separate follow-ups.

References #427, acceptance rule 6. This PR is the remaining Nicely record of the delivery.

Review state: ready. All accepted findings are fixed; exact-head signoff and the hosted Check job passed.

Four independent Opus 5.5 xhigh reviews ran in two pairs. The first pair reviewed 81c5cb59; its three prose findings are fixed in 3d99d919. The final pair reviewed 3d99d919; its accepted clarity and ledger-link findings are fixed in 6aaebd34. Follow-up-list parallelism was noted as optional style. The final small edits passed direct source/link checks, plan/doc tests, and limits, with no further delegate requested.

The complete local check passed. Initial signoff on 6aaebd341814e1bb90cc50464e478fc077b55609 hit the existing #477 timing flake. The unchanged-head focused rerun passed:

Initial just signoff, exit 1:
signal_test.go:35: stream-holding child delayed timeout to 26.496600733s, want less than 25s

go test -count=1 -run '^TestSignalTimeoutBoundsStreams$' -v ./sdk/extensiontest, exit 0:
signal_test.go:35: timeout returned after 21.612823987s: exit status 1
--- PASS: TestSignalTimeoutBoundsStreams (21.61s)
PASS

The nested fixture's exit 1 is the expected negative control. The first signoff failed only its outer elapsed bound. No source or timeout changed. The complete recipe retry on the same SHA passed, exit 0:

Signed off on 6aaebd341814e1bb90cc50464e478fc077b55609

Its detached receipt records just check exit 0 and recipe signoff exit 0. GitHub reports signoff: success for that SHA. The hosted Check job also passed on that SHA; CodeRabbit skipped its automatic review. The initial failing run posted no signoff.

Post-merge owner: Chef 4. Manual issue closure covers #427 and Epic 15 (#455), followed by milestone COR-104. Milestone COR-102 is already open. No out-of-run issues move into it.

  • 1. Spec: n/a, no command, flag, JSON key, or error code changes
  • 2. Scenario: n/a, documentation and backlog state only
  • 3. Code: n/a, no executable changes
  • 4. Text: n/a, no CLI catalog strings
  • 5. Checks: uv run --no-project --python 3.12 just check, exit 0
  • 6. Verify: n/a under verify-nicely, no affected real CLI feature
  • 7. Docs: card, queue, status, and D068 updated, with delivery links verified
  • 8. Sign off: uv run --no-project --python 3.12 just signoff, retry exit 0 on 6aaebd341814e1bb90cc50464e478fc077b55609

by GPT-6.1-Sol via Codex

Record the video-convert adoption evidence for #427 acceptance rule 6
Mark COR-104 Shipped and advance the queue to COR-102-T1
Keep UAT, gate removal, and the five follow-ups pending
@pascalandy pascalandy self-assigned this Nov 4, 2026
@coderabbitai

coderabbitai Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Repository UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: b885acb6-b12e-4184-b5ed-ba3cd57aed19

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Retain the dated D068 starting point beside the adoption outcome
Remove incorrect run attribution and the repeated UAT status
Name the external PR and the extension that adopted the loop
Link correction records without copying extension-owned ledger rows
@pascalandy
pascalandy merged commit 6aaebd3 into main Nov 4, 2026
3 checks passed
@pascalandy
pascalandy deleted the cor-104-t10-done-1 branch November 4, 2026 20:03
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