Skip to content

Add fast, integration, browser, and platform test lanes - #67

Merged
itsjling merged 3 commits into
mainfrom
codex/issue-036-release-proof
Jul 31, 2026
Merged

Add fast, integration, browser, and platform test lanes#67
itsjling merged 3 commits into
mainfrom
codex/issue-036-release-proof

Conversation

@itsjling

@itsjling itsjling commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Closes #36

Summary

  • split quick, integration, coverage, browser, and host-shell checks into named lanes
  • enforce per-file coverage floors for the core parser, target, note, and presenter paths
  • run pull request lanes on Linux and check macOS and Windows shell rules on a set schedule
  • keep Chromium setup separate from the browser test command

Stack and blockers

Verification

  • npm run lint
  • npm test (35 unit, 40 integration, 37 coverage, and 2 browser tests)
  • npm run test:platform (2 host-shell tests)
  • npm run docs:check
  • npm run docs:build
  • npm run fallow:audit -- --base origin/codex/issue-028-browser-journey
  • git diff --check

Stack

  1. Make the fresh-checkout demo self-contained #42
  2. Add one-command live and mocked development modes #58
  3. Prove one full review journey in a real browser #59
  4. Add fast, integration, browser, and platform test lanes #67 👈 current

@itsjling

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: 5ce5ff5ce9

ℹ️ 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 tests/present-agent.test.mjs Outdated
@itsjling
itsjling force-pushed the codex/issue-028-browser-journey branch from d47489e to 89f965e Compare July 30, 2026 18:00
@itsjling
itsjling force-pushed the codex/issue-036-release-proof branch from 5ce5ff5 to a9f256a Compare July 30, 2026 18:37
@itsjling

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

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

Reviewed commit: a9f256a7f6

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

@itsjling
itsjling force-pushed the codex/issue-028-browser-journey branch from 89f965e to ba077fd Compare July 30, 2026 18:57
@itsjling
itsjling force-pushed the codex/issue-036-release-proof branch from a9f256a to ee26ce4 Compare July 30, 2026 19:05
@itsjling
itsjling force-pushed the codex/issue-028-browser-journey branch from ba077fd to 0241f65 Compare July 30, 2026 19:32
@itsjling
itsjling force-pushed the codex/issue-036-release-proof branch from ee26ce4 to 72ee69a Compare July 30, 2026 19:32
@itsjling
itsjling force-pushed the codex/issue-036-release-proof branch from 72ee69a to 735e034 Compare July 30, 2026 19:45
@itsjling
itsjling changed the base branch from codex/issue-028-browser-journey to main July 31, 2026 05:50
@itsjling itsjling added the automation-reviewed Repo-owned automation changes were reviewed label Jul 31, 2026
@itsjling
itsjling merged commit 36111ed into main Jul 31, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation-reviewed Repo-owned automation changes were reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add fast, integration, browser, and platform test lanes

1 participant