Skip to content

Add opencode command for creating compat flag docs PR#6519

Merged
jasnell merged 1 commit into
mainfrom
jasnell/opencode-command-to-create-compat-flag-doc-pr
Apr 21, 2026
Merged

Add opencode command for creating compat flag docs PR#6519
jasnell merged 1 commit into
mainfrom
jasnell/opencode-command-to-create-compat-flag-doc-pr

Conversation

@jasnell

@jasnell jasnell commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@jasnell jasnell requested review from danlapid, guybedford and npaun April 7, 2026 23:48
@jasnell jasnell requested review from a team as code owners April 7, 2026 23:48

@ask-bonk ask-bonk Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm Bonk, and I've done a quick review of your PR.

Adds an opencode command for automating compat flag documentation PRs in cloudflare-docs.

  1. [MED] Line 24 references .github/workflows/compat-flag-docs.yml but that file does not exist in the repo. If this workflow hasn't been merged yet, a note clarifying that would help; otherwise the path may be wrong.
  2. [LOW] Typo on line 118: "present the draft changes to the use" should be "to the user".
  3. [LOW] Line 129 has garbled backtick/quoting around the gh pr view invocation.

This review was generated by an AI code review assistant and may contain inaccuracies.

Comment thread .opencode/commands/compat-flag-docs.md
Comment thread .opencode/commands/compat-flag-docs.md
Comment thread .opencode/commands/compat-flag-docs.md
@ask-bonk

ask-bonk Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Review posted on PR #6519. Three findings:

  1. [MED] Line 24 references .github/workflows/compat-flag-docs.yml which doesn't exist in the repo -- either the path is wrong or it's a planned addition that should be noted.
  2. [LOW] Typo on line 118: "to the use" → "to the user" (suggestion provided).
  3. [LOW] Broken backtick nesting on line 129 around the gh pr view command (suggestion provided).

github run

@harrishancock harrishancock left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it should ideally also:

  • Update any relevant runtime API documentation section.
  • Add a changelog entry.

@jasnell

jasnell commented Apr 21, 2026

Copy link
Copy Markdown
Collaborator Author

Let's tackle those as a follow up

@jasnell jasnell merged commit 7872c70 into main Apr 21, 2026
27 checks passed
@jasnell jasnell deleted the jasnell/opencode-command-to-create-compat-flag-doc-pr branch April 21, 2026 16:56
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.

2 participants