Skip to content

Document --json flag for pages project list command#28175

Merged
petebacondarwin merged 1 commit intoproductionfrom
devin/1770485560-pages-project-list-json
Apr 20, 2026
Merged

Document --json flag for pages project list command#28175
petebacondarwin merged 1 commit intoproductionfrom
devin/1770485560-pages-project-list-json

Conversation

@petebacondarwin
Copy link
Copy Markdown
Contributor

@petebacondarwin petebacondarwin commented Feb 7, 2026

Summary

Documents the new --json flag for wrangler pages project list in the Direct Upload guide. This corresponds to cloudflare/workers-sdk#12465, which adds JSON output support to the command for programmatic use and scripting workflows.

The auto-generated command reference (via WranglerNamespace) will automatically pick up the --json flag once the wrangler dependency is bumped to include the change.

Link to Devin run: https://app.devin.ai/sessions/b4459c91b7b74c93b13ab3c365b05197
Requested by: @petebacondarwin

Documentation checklist


Open with Devin

@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 7, 2026

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/pages/ @cloudflare/workers-docs, @GregBrimble, @WalshyDev, @aninibread, @irvinebroque, @cloudflare/pcx-technical-writing, @cloudflare/product-owners

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 7, 2026

@elithrar
Copy link
Copy Markdown
Collaborator

elithrar commented Feb 7, 2026

/bonk review for Pete. Confirm the flag usage and use the linked commit to validate.

@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented Feb 7, 2026

I've now reviewed both the docs PR and the linked workers-sdk commit. Here's my review:


Review Summary

Flag usage: Confirmed. The --json flag documented here matches the implementation in workers-sdk#12465 (commit 7bae777). The implementation adds a json arg of type boolean (default false) to pagesProjectListCommand, and when set, outputs JSON.stringify(data, null, 2) instead of a formatted table. The docs PR correctly shows:

npx wrangler pages project list --json

This is exactly what the code accepts.

What looks good:

  • Placement is logical -- directly after the existing pages project list example, building on it naturally.
  • The description ("for programmatic use or scripting") aligns with the PR motivation and the changeset description.
  • The PR body correctly notes that the auto-generated command reference (via WranglerNamespace) will pick up the flag once the wrangler dependency is bumped.

One minor observation: The workers-sdk PR is still open (not yet merged). This docs PR will be accurate once that lands and the wrangler dependency is bumped. No issue if this docs PR merges first -- the guidance will just be slightly ahead of the released CLI, which is standard practice.

No changes needed. This is clean and correct. LGTM.

github run

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 3, 2026

Hey there, we've marked this pull request as stale because there's no recent activity on it. This label is helps us identify PRs that might need updates (or to be closed out by our team if no longer relevant).

@github-actions github-actions Bot added the stale label Mar 3, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2026

Hey there, we've closed out this pull request because it's been stale for a while and there's been no additional action on it. If these changes are still relevant, open a new pull request (or flag to us in a GitHub issue).

@github-actions
Copy link
Copy Markdown
Contributor

Hey there, we've marked this pull request as stale because there's no recent activity on it. This label helps us identify PRs that might need updates (or to be closed out by our team if no longer relevant).

@petebacondarwin
Copy link
Copy Markdown
Contributor Author

Not stale - waiting for review

Co-Authored-By: pbacondarwin@cloudflare.com <pete@bacondarwin.com>
@petebacondarwin petebacondarwin force-pushed the devin/1770485560-pages-project-list-json branch from e93e619 to 524f98f Compare April 17, 2026 22:05
@petebacondarwin petebacondarwin merged commit d817e90 into production Apr 20, 2026
13 checks passed
@petebacondarwin petebacondarwin deleted the devin/1770485560-pages-project-list-json branch April 20, 2026 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants