Skip to content

fix: make Delivery and GitHub status cards truthful#66

Merged
Codename-11 merged 13 commits into
mainfrom
codex/axi-117-delivery-github-truth
Jul 17, 2026
Merged

fix: make Delivery and GitHub status cards truthful#66
Codename-11 merged 13 commits into
mainfrom
codex/axi-117-delivery-github-truth

Conversation

@Codename-11

Copy link
Copy Markdown
Owner

Implements AXI-117.

Summary

  • derives and persists native IMPLEMENTS, FIXES, RELEASES, and RELATES_TO GitHub relations without duplicate cards
  • reconciles terminal PR metadata so merged state cannot conflict with pending checks or unknown mergeability
  • separates Delivery actor, invocation source, connector/transport, and actual dispatched runtime evidence
  • uses semantic success styling for passing Ready to Close evidence and keeps detailed evidence expandable

Verification

  • pnpm lint (passes; existing warnings only)
  • pnpm typecheck
  • pnpm test --no-file-parallelism (179 files passed, 1 skipped; 1399 tests passed, 1 skipped)
  • production Next.js build
  • targeted issue-flow Playwright test
  • full Playwright suite: 51 journeys passed before one Chromium process crash; the two affected Mission Control journeys passed immediately in isolation, for all 53 journeys passing across the run and retry
  • fresh local Postgres migration and pnpm prisma validate

pnpm ci:local was also invoked, but its WSL launcher selected Linux Node with the Windows node_modules tree and could not load Rollup's Linux optional native package. Every constituent gate was run natively and passed as reported above.

No merge, release, tag, deployment, or production mutation is included.

@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: 0369af7b75

ℹ️ 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/server/services/github/resource-sync.ts
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

@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: b45834f374

ℹ️ 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/server/services/github/reconciliation.ts Outdated
Comment thread src/server/services/github/relation.ts Outdated
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

@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: 86f6b08874

ℹ️ 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/server/services/github/resource-sync.ts Outdated
Comment thread src/server/services/github/resource-sync.ts
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

@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: 6d8d96ab4e

ℹ️ 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/server/services/github/webhook.ts
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Bravo.

Reviewed commit: 46a87ce308

ℹ️ 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".

@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

@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: b371462633

ℹ️ 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 prisma/migrations/20260716200000_delivery_github_relation_truth/migration.sql Outdated
Comment thread src/server/services/github/resource-sync.ts
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

@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: 7449591d0f

ℹ️ 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/server/services/github/resource-sync.ts
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

Please review the exact release candidate head fa41fb9, including the scoped Playwright correction.

@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: fa41fb9fbe

ℹ️ 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/server/services/github/relation.ts
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

Please review exact head aa2d4ee after addressing all four unresolved relation-identity and completion-safety findings.

@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: aa2d4eee63

ℹ️ 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/server/services/github/resource-sync.ts Outdated
Comment thread src/server/services/github/resource-sync.ts
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

Please review exact head d3088de. This adds regression-covered SOURCE preservation during canonicalization and keeps imported SOURCE PRs eligible for mapped status sync.

@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: d3088de44c

ℹ️ 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/server/services/github/resource-sync.ts
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

Please review exact head b204d9b. This closes the migration and canonicalization stale-completion cleanup paths with focused regression coverage.

@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: b204d9b9a8

ℹ️ 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 prisma/migrations/20260716200000_delivery_github_relation_truth/migration.sql Outdated
Comment thread src/components/markdown/attachment-renderer.tsx Outdated
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review

Please review exact head e9db3f5 after the final migration-order and bare-URL tokenization fixes.

@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: e9db3f5137

ℹ️ 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/server/services/github/webhook.ts Outdated
Comment thread src/components/markdown/attachment-renderer.tsx Outdated
@Codename-11

Copy link
Copy Markdown
Owner Author

@codex review the exact current head 389642e. Please re-check the complete diff for correctness, integrity, tenant isolation, reconciliation symmetry, and regressions. The latest patch includes imported SOURCE failed-check status synchronization and URL-before-compact-reference tokenization.

@Codename-11
Codename-11 merged commit 684df53 into main Jul 17, 2026
3 checks passed
@Codename-11
Codename-11 deleted the codex/axi-117-delivery-github-truth branch July 17, 2026 01:02
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