Skip to content

fix(metadata): accept complete catalogs with stale summary counts - #172

Closed
DeusExTaco wants to merge 1 commit into
developfrom
feature/v2.0-resume
Closed

DeusExTaco wants to merge 1 commit into
developfrom
feature/v2.0-resume

Conversation

@DeusExTaco

Copy link
Copy Markdown
Contributor

Summary

  • Separate advisory series summary counts from complete, stable paginated issue catalogs.
  • Preserve original provider evidence and derive saved issue counts from validated membership.
  • Keep final Add revalidation, identity/parent checks, resource limits, and atomic failure behavior.

Validation

  • Reproduced the stale-count Add HTTP 409 and unnecessary Refresh fallback before the fix.
  • make validate: 15,899 passed, 1,593 skipped, 1 expected failure; lint, format, strict typing and CSS build passed.
  • Separate disposable SQLite/PostgreSQL parity run: 270 passed.
  • Normal commit hooks passed.

Boundaries

No schema migration, archive mutation, import/recovery rewrite, release, or production deployment. Safe linked-catalog extension is the next separately tested checkpoint. Keep this PR open while the approved v2.0 reliability checkpoints are qualified.

@DeusExTaco DeusExTaco added the ci:full Run the full CircleCI PR gate label Oct 16, 2026 — with ChatGPT Codex Connector

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a20b99a1fd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/pullbox/services/metadata_series_adoption.py
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-16T21:05:38.969007Z a20b99a PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@DeusExTaco DeusExTaco closed this Oct 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:full Run the full CircleCI PR gate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant