Skip to content

feat: add evidence-bound product semantic catalog - #692

Merged
seonghobae merged 11 commits into
fix/ask-auto-source-composerfrom
feat/product-semantic-catalog
Aug 26, 2026
Merged

feat: add evidence-bound product semantic catalog#692
seonghobae merged 11 commits into
fix/ask-auto-source-composerfrom
feat/product-semantic-catalog

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Outcome

  • adds normalized product group/model/variant/trade-item catalog identities with scoped GTIN/MPN keys
  • validates contextual-orchestrator structured product spans and preserves unique/tie/missing/unavailable resolution
  • reuses the durable post-content worker with exact authorized-input digest idempotency
  • exposes authorized product evidence in Post detail and adds SHACL/Storybook states

Evidence

  • backend focused: 66 passed
  • frontend focused/i18n: 65 passed
  • new Python modules: 100% targeted coverage
  • frontend lint: pass
  • Storybook production build: pass
  • rendered catalog-review-required story inspected at 1280x720

Stacked on #688; no canonical runtime mutation.


Open in Devin Review

@coderabbitai

coderabbitai Bot commented Aug 26, 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: ebef3d13-a759-4b22-a15b-883d3467f132

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

Exact-head audit found and fixed an authorization gap at d8c3251: a visible post could previously return a product span whose evidence_post_id referred to a private post outside the reader scope. The post-detail query now applies the shared source eligibility predicate and account ABAC to every evidence post before returning the mention. A real PostgreSQL/OIDC regression inserts a hidden cross-entity evidence post and proves it is omitted. Product-focused integration/unit checks: 10 passed. The broader documentation pin failure on this stack is inherited from the parent divergence now being reconciled in #693, not suppressed here.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@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

Comment thread backend/app/post_content_worker.py
@seonghobae
seonghobae merged commit cdd499b into fix/ask-auto-source-composer Aug 26, 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.

1 participant