Skip to content

test(lastcode): restore browser and repair permission fixtures - #316

Merged
lastobelus merged 2 commits into
lastcode/mainfrom
lastcode/2752-browser-capability-fixtures
Oct 7, 2026
Merged

lastobelus merged 2 commits into
lastcode/mainfrom
lastcode/2752-browser-capability-fixtures

Conversation

@lastobelus

@lastobelus lastobelus commented Oct 7, 2026 •

Copy link
Copy Markdown
Owner

Full validation fails in the saved-handoff and markdown link fixtures because their environment mocks omit the server-browser capability lookup. The repair permission test also expects the full-access requirement to run before the target runtime-mode guard.

Advertise server-browser support in both web fixtures and exercise both repair permission barriers with matching target modes. Rejected requests must never launch a repair; a full-access client still launches exactly once.

Validation: all 94 tests in the two web files and all 15 server toolkit tests pass; targeted lint, formatting, and diff checks pass. GitHub CI provides the full validation.

Implemented by gpt-6.1-sol using the Codex harness.

Carry-Group: incubator
Carry-Observation: Handoff and markdown fixtures must advertise server-browser support to exercise preview opening after the native browser integration.
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 7, 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-07T07:09:54.960478Z e5c4506 Manual request
ℹ️ 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.

@lastobelus

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

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

Reviewed commit: 8f6ced5638

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

Carry-Group: incubator

Carry-Observation: Repair tests distinguish the target runtime-mode guard from the full-access repair requirement and verify rejected requests never launch a repair.
@lastobelus lastobelus changed the title test(web): restore handoff and markdown preview fixtures test(lastcode): restore browser and repair permission fixtures Oct 7, 2026
@lastobelus

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: e5c4506b06

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

@lastobelus
lastobelus merged commit cf10846 into lastcode/main Oct 7, 2026
19 of 34 checks passed
@lastobelus
lastobelus deleted the lastcode/2752-browser-capability-fixtures branch October 7, 2026 07:16
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