Skip to content

test: verify retention evidence invalidation - #3

Merged
seonghobae merged 3 commits into
codex/policyweave-focus-evidencefrom
codex/policyweave-retention-evidence
Sep 9, 2026
Merged

test: verify retention evidence invalidation#3
seonghobae merged 3 commits into
codex/policyweave-focus-evidencefrom
codex/policyweave-retention-evidence

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Stack

  • base: codex/policyweave-focus-evidence@b38204b805291c2d3cded252dafc226ea5b35873 (PR #2)
  • exact head: 1b3d34bb8f98ed240cccc36f9ea4b05ca3b1c68f
  • verification-only successor; no production behavior is changed

Browser contract

Across desktop, tablet, and mobile Chromium, the operator moves to retention with the keyboard, records 보유함 plus a period, switches to explicit 보유하지 않음, and switches back. The test requires:

  • the correct heading receives focus
  • applies + period marks the responsibility complete and renders the period
  • switching to none removes the dependent period input, remains complete, and renders the explicit negative fact
  • switching back to applies does not revive the stale period, blocks completion, and renders the missing-period state

Exact-head evidence

  • initial hosted CI 34081110910 failed before product interaction because the broad locator matched both rail and warning buttons; production was not changed
  • 53f109a scoped navigation to the authoring rail and established 12/12 browser GREEN
  • 1b3d34b records the evidence in CHANGELOG, TRD, and the product/technical gap baseline
  • CI 34081414494: terminal success; clean install, 0 dependency vulnerabilities reported, lint, Vitest 46/46, production build, Playwright/axe 12/12
  • artifact 10003815535: sha256:031c8689effb52e92fdab117b993c1e4a8b58b19b6209b6ef7d8724e8bfe6f81
  • independent approval: pending

Ready status admits exact-head review; it does not assert merge readiness.

@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 40 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 4b879882-ed5d-47cd-8d23-a868c58f7613

📥 Commits

Reviewing files that changed from the base of the PR and between b38204b and 1b3d34b.

📒 Files selected for processing (4)
  • CHANGELOG.md
  • docs/TRD.md
  • docs/product-technical-gap-baseline.md
  • tests/e2e/authoring.spec.ts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae
seonghobae marked this pull request as ready for review September 7, 2026 03:59

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

Please review exact head 1b3d34bb8f98ed240cccc36f9ea4b05ca3b1c68f. The locator-only RED and current 12/12 browser GREEN are bound in the PR body.

@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown

@seonghobae I will review pull request #3 at exact head 1b3d34bb8f98ed240cccc36f9ea4b05ca3b1c68f.

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant