Skip to content

feat: check and resume Linear publication - #486

Open
mldangelo-oai wants to merge 2 commits into
mainfrom
mdangelo/codex/publication-preflight-and-retry
Open

feat: check and resume Linear publication#486
mldangelo-oai wants to merge 2 commits into
mainfrom
mdangelo/codex/publication-preflight-and-retry

Conversation

@mldangelo-oai

@mldangelo-oai mldangelo-oai commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add an explicit way to inspect publication readiness and retry a partially published scan without recreating issues that are already recorded locally. The existing publish-all, create-new default is unchanged.

Searched existing PRs and issues before starting; no matching readiness/retry fix or tracked issue was found. This is independent of the receipt-verification and selection work in #482 and #484.

Changes

  • Share one read-only SQLite lookup for the exact scan occurrence, Linear team, and optional project. Read committed WAL entries without creating or migrating scan history, and forward cancellation through Python discovery and the workbench subprocess.
  • Add publish scan --skip-existing and SDK skipExisting. Keep full-scan history validation, publish only the remaining findings, and report recorded successes separately as skipped.
  • Add publish check and SDK checkScanPublication. With an API key, use only read queries to check authentication, team, project membership, and an optional assignee. Do not upload finding descriptions or create a test issue.
  • Document the limits: connected-app access is not checked, issue-creation permission is not tested, and unrecorded or concurrent remote operations are not made exactly once.

Testing

  • Seven-file publication suite at 4aa50c13: 164 passed, 0 failed, 1466 assertions.
  • pnpm run types, pnpm run format, and git diff --check: passed.
  • Fresh package build, package allowlist, and installed CLI/SDK smoke: passed, 218 package entries.
  • Real SQLite-backed synthetic tests cover partial retries, exact destination matching, read-only old history, committed WAL entries, unchanged default republish behavior, and preservation of sealed artifacts.
  • Completed local PR review, deslop PR review, and code simplification passes. No remaining actionable local findings.
  • Full and randomized suites (seed 12345) at 4aa50c13: each passed 1314 tests, skipped 11, failed 0, with 9287 assertions.

Risk and rollout

Additive and opt-in; no schema migration or dependency change. The skip decision uses recorded local associations, not a remote issue search. Recover retained handoffs before retrying uncertain operations. Concurrent publishers can still race. Read-only access checks do not establish write permission. All tests use isolated synthetic data and injected clients; no live Linear issues are created.

Public disclosure review

  • No customer, partner, prospect, or user identities, data, or identifying details are included.
  • No credentials, personal data, private source, scan findings, or nonpublic links or tickets are included.
  • I reviewed the branch name, title, description, commits, changes, comments, logs, screenshots, attachments, and links for public disclosure.

@mldangelo-oai

Copy link
Copy Markdown
Collaborator Author

@codex review Please review exact head 6b47211, especially read-only preflight behavior, exact history matching, and partial-retry accounting.

@github-actions github-actions Bot added the enhancement New feature or request label Aug 16, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Swish!

Reviewed commit: 6b47211542

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

@chatgpt-codex-connector

Copy link
Copy Markdown

Security review completed. No security issues were found in this pull request.

Reviewed commit: 6b47211542

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

@mldangelo-oai

Copy link
Copy Markdown
Collaborator Author

@codex review The full and randomized suites are now green. Please review exact head 6b47211.

@chatgpt-codex-connector

Copy link
Copy Markdown

Security review completed. No security issues were found in this pull request.

Reviewed commit: 6b47211542

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

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

ℹ️ 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 sdk/typescript/src/publish.ts Outdated
@mldangelo-oai

Copy link
Copy Markdown
Collaborator Author

@codex review Please review exact head 4aa50c1. The history-inspection cancellation feedback is fixed additively. Focused publication tests, types, formatting, and fresh installed-package smoke pass; full-suite verification is running.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Another round soon, please!

Reviewed commit: 4aa50c13af

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

@chatgpt-codex-connector

Copy link
Copy Markdown

Security review completed. No security issues were found in this pull request.

Reviewed commit: 4aa50c13af

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

@mldangelo-oai
mldangelo-oai marked this pull request as ready for review August 16, 2026 09:43
@mldangelo-oai

Copy link
Copy Markdown
Collaborator Author

@codex review The validation record is complete and the PR is ready. Please review exact head 4aa50c1. Full and randomized suites each passed 1,314 tests, the installed-package smoke passed, and all current-head GitHub checks are green.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

Reviewed commit: 4aa50c13af

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

@chatgpt-codex-connector

Copy link
Copy Markdown

Security review completed. No security issues were found in this pull request.

Reviewed commit: 4aa50c13af

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant