Skip to content

docs: refresh current product gap and PR gate audit - #325

Merged
seonghobae merged 31 commits into
fix/validate-partial-image-regionsfrom
docs/current-gap-audit
Aug 21, 2026
Merged

docs: refresh current product gap and PR gate audit#325
seonghobae merged 31 commits into
fix/validate-partial-image-regionsfrom
docs/current-gap-audit

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Refresh the product-gap baseline with the current exact stacked heads and
    gate states.
  • Preserve semantic footnote units for HTML, Word MsoFootnoteText, Word
    footnote links, and OOXML w:footnote/w:endnote containers.
  • Keep the evidence boundary explicit: branch and hosted-Checks evidence is
    not protected-main release evidence.

Dependency

  • Current parent fix: validate retained visual locator regions #324:
    8bcfea8d91810323a50dff89ba0d2e74170928e2.
  • Current head:
    d80d691cd8ed780a8fb0072199304f595b1ea75e after an ordinary parent
    restack and preservation of the concurrent docs branch updates.

Verification

  • uv run --locked --extra dev --extra backend python -m pytest -q
    744 passed, 17 skipped, 4 warnings.
  • uv run --locked --extra dev --extra backend python -m pytest -q tests/test_chunking.py
    36 passed.
  • actionlint and git diff --check passed.
  • Synthetic fixtures only; no credentials or real records are included.

This PR has no frontend source change; the inherited #324 frontend gate
remains required. Merge only after the exact current head has independent
approval and terminal Checks.

@seonghobae

Copy link
Copy Markdown
Contributor Author

@opencode-agent review exact current HEAD 80dc0ab. Verify that the product-gap baseline current-head table is faithful to the live PR base/head/review/check state and does not turn branch evidence into a protected-main claim; publish a formal review.

@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: fe1be551-6f5c-4bd5-b2b0-693ec453e91d

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

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.

devin-ai-integration[bot]

This comment was marked as resolved.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Current exact head is now 6478994807ac9cb944c34b2240d81b55245fe182. Please supersede the prior review target and review this commit against base fix/validate-partial-image-regions. Required verdict: findings or explicit no-findings, with Checks considered only for this exact head.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Base synchronization and traceability fix are now on exact HEAD 39acad39bfdf462bf6734875d70c048a91a1a258. The earlier dangling-ADR finding is addressed: this branch now states that ADR 0118 is proposed in PR #322 and is not claimed as present here. Please review this exact head against current base; publish findings or explicit no-findings.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Exact current HEAD is now d2c20380c3ee95f2a800bde901a1e11917e7f977; the audit also records latest #322/#324 heads. Please supersede prior review targets and review only this exact head against current base.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Base is synchronized to current #324 head fdd62a6f8317c93f9ba5fc27393cfb26c69e584a. The audit row now records #322 successful hosted Checks and the current #324 head. Exact current HEAD is 0d5538228dfec21cba51ee48dcf64cb7d2545b3e; please review only this head against current base and publish findings or explicit no-findings.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Correction: the exact current HEAD is 0d5538220830eab6cd2823dd67ff8113ffcbf945. Base is synchronized to #324 head fdd62a6f8317c93f9ba5fc27393cfb26c69e584a; review only this exact head against current base and publish findings or explicit no-findings.

devin-ai-integration[bot]

This comment was marked as resolved.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Validation checkpoint: exact current HEAD is d86af3bd3456328e7828047efdc4d3e3e6aa471f (pushed from docs/current-gap-audit). Full backend suite: 741 passed, 16 skipped, 4 warnings. Focused chunking/normalization/persistence suite: 62 passed; ruff and compileall passed. This checkpoint preserves semantic HTML/Word footnote units (role="doc-footnote", footnote containers, MsoFootnoteText, Word reference links) for persisted buyer rendering and search. Please review this exact HEAD; the branch remains pending independent approval and required Checks.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Follow-up validation checkpoint: exact current HEAD is d4408c67a563244f3595ec77d82c71d727eaf21a (the OOXML container addition and citation-vs-definition guard are both included). Full backend suite: 742 passed, 17 skipped, 4 warnings; focused semantic/persistence suite: 64 passed; ruff, compileall, and diff checks passed. Devin's body-citation overmatch finding is fixed with a regression test. Please review this exact HEAD; independent approval and required Checks remain pending.

@seonghobae

Copy link
Copy Markdown
Contributor Author

@opencode-agent please review exact current HEAD 704d32e.

This update records only aggregate local Compose/PostgreSQL evidence in docs/product-technical-gap-baseline.md: source_post 43,839; post_summary_role 179; post_summary_person_mention 17; post_summary_action 88; requester 30; processor 45; evidence 88; project-bound 85. No source text or identifiers were exported.

The commit is based on the concurrently advanced d4408c6. Please re-run the required checks and provide an independent formal review for this exact head. Do not treat aggregate local evidence as protected-main or release evidence.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Exact-head follow-up: the branch now includes the remote runtime aggregate evidence checkpoint and the baseline points to 5c5b71cfd22d7d90b2d5480cf9f074ddfb0f57da. The semantic footnote fix remains unchanged and was revalidated before the evidence-only update (742 passed, 17 skipped). Please review this exact HEAD; the PR is still blocked pending required Checks and independent approval.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Audit alignment checkpoint: exact current HEAD is 6d32e8d8198c0de7d6536a3813ab70772f2a86d8. The baseline now records #322 exact HEAD 1126cfa026876d2427a6f6cf6001eaa8ac609ad5 and its per-batch embedding provenance guard, while retaining the footnote citation and OOXML checkpoints. This is documentation-only after the previously validated code changes; required Checks and independent approval remain pending.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Review exact current HEAD d80d691 after concurrent stack updates. The documented fresh aggregate runtime evidence remains present at docs/product-technical-gap-baseline.md: source_post 43,839; roles 179; person mentions 17; actions 88; requester 30; processor 45; evidence 88; project-bound 85. No source identifiers/text were exported.

Required Checks are queued again. Please provide an independent formal review on this exact SHA after revalidation; this remains a docs PR and is not merge/release evidence until the protected stack gate is satisfied.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Exact-head documentation update (head 9dc8c7d143841a1e93459d0a5d35d6332184be0c): added the anonymized buyer-reported gap matrix to docs/product-technical-gap-baseline.md, mapping footnotes/lists, malformed tables, indentation, mixed projects/5W1H, image OCR/regions/Markdown, aliases/SearXNG, PM affiliation, R&R/requester/processor/DAG, payment evidence, and superscript/subscript semantics to ADRs and current PRs. The section explicitly separates branch-local evidence from protected-main claims and preserves the APA-7/research-register boundary. git diff --check passed. No independent formal approval is present and required Checks remain queued; PR remains gated and is not being merged.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Exact-head refresh: pushed 19b4958ea99b20dc7a4fa60e44e352c2b3a8767d, adding the current-loop evidence table for PRs #303, #323, #325, and #340 with local results and protected-gate state. This preserves the buyer-gap mapping and does not promote any branch result to protected-main evidence. Approval is still absent and required Checks remain queued.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Exact-head refresh: baseline now also records the newly stacked buyer-gap PR #344 at 922a38405e3f89779a0a70974a6ad1f8f2bb4793 with its focused backend/frontend evidence and queued protected gate. Documentation branch head is 64ce5fbac4ed9c6233ecaab475d6702080a8e7e2; no release or protected-main claim is made.

@seonghobae
seonghobae enabled auto-merge (squash) August 21, 2026 05:04
@seonghobae seonghobae added the documentation Improvements or additions to documentation label Aug 21, 2026 — with ChatGPT Codex Connector
…ns' into repair/pr325-restack

# Conflicts:
#	docs/product-technical-gap-baseline.md

@devin-ai-integration devin-ai-integration 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.

Devin Review found 1 new potential issue.

Open in Devin Review

| #340 Naruon provider contract | `1e792a761f96e2184394a15f112cc947c7661c41` | Contract tests `11 passed`; module coverage `100%`; compileall and diff check passed | No approval; required Checks queued |
| #344 metric script semantics | `922a38405e3f89779a0a70974a6ad1f8f2bb4793` | Backend focused `74 passed`; frontend focused `29 passed`, lint, and Vite build passed | Stacked on #303; no approval; required Checks queued |

These four rows are exact-head observations from the current review loop. They

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Row count in gate snapshot summary is wrong

The gate snapshot table lists five PRs (#303, #323, #325, #340, #344), but the summary sentence calls them "These four rows." The count contradicts the table above it.

Suggested change
These four rows are exact-head observations from the current review loop. They
These five rows are exact-head observations from the current review loop. They
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

* docs: record current exact-head gap checkpoint

* docs: retain current buyer stack roots

* docs: record current exact-head review gates

* docs: refresh current queue roots

* docs: record buyer-stack restack

* docs: record MCP key boundary closure
@seonghobae
seonghobae merged commit 716003f into fix/validate-partial-image-regions Aug 21, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant