Repository navigation
docs(sites): correct the command reference and agent skill - #249
Conversation
🦋 Changeset detectedLatest commit: 9bd040f The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8faca51c33
ℹ️ 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".
| - Pass `--force` on anything with a confirmation (publish, prune, remove, delete, `create --from-zone`); without a TTY they error with a hint rather than waiting on a prompt. | ||
| - Pass the site explicitly (or commit `bunny.jsonc` with `sites.name`); the interactive picker is disabled under `--output json` and by `--force`, so `sites delete --force` with nothing linked errors instead of prompting. | ||
| - `--output json` on every command emits machine-readable results. `deploy` prints `{ id, production, unchanged, live }`, where `production` is `null` on a site whose hostname couldn't be read. | ||
| - `--output json` on every command emits machine-readable results. `deploy` prints `{ site, id, source, files, bytes, production, unchanged, live }`, where `production` is `null` on a site whose hostname couldn't be read. |
There was a problem hiding this comment.
Account for unchanged deploys in the documented JSON shape
When sites deploy --output json redeploys content that is already live, the handler returns early with only site, id, unchanged, live, and production; source, files, and bytes are absent. This skill now promises those fields for every deploy result, so agents can mis-handle this common no-op response; document the conditional fields or make the payload consistent.
Useful? React with 👍 / 👎.
| | `--previous` | `deployments publish` | Publish the previous deploy (instant rollback) | | ||
| | `--keep` | `deployments prune` | Number of recent deploys to keep (default 5; live and previous are always kept) | | ||
| | `--ssl`, `--wait`, `--force-ssl` | `domains add` | Issue SSL now; wait up to 10 minutes for DNS then issue it; `--no-force-ssl` keeps HTTP working | | ||
| | `--force-ssl` | `ssl`, `domains ssl` | Force HTTP→HTTPS on the system host; `--no-force-ssl` allows plain HTTP | |
There was a problem hiding this comment.
Describe domains SSL as acting on the custom domain
For bunny sites domains ssl <domain>, --force-ssl controls HTTP-to-HTTPS redirection on the named custom domain; only the top-level bunny sites ssl command operates on the system hostname. Adding domains ssl to this row while retaining “on the system host” misdocuments the command's target, so split the entries or generalize the description.
AGENTS.md reference: AGENTS.md:L3-L5
Useful? React with 👍 / 👎.
|
Corrects the
bunny sitescommand reference and agent skill.