Skip to content

fix: treat a named request as consent to force #75

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
AntTheLimey wants to merge 3 commits into
base: main
Choose a base branch
from
Open

fix: treat a named request as consent to force #75

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2e48387
fix: treat a named request as consent to force
AntTheLimey Oct 21, 2026
e93bf30
fix: load the managed skill for a bare database name
AntTheLimey Oct 21, 2026
f101748
revert: drop the bare-name managed skill trigger
AntTheLimey Oct 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions plugins/pgedge/skills/pgedge-byoc/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -108,9 +108,11 @@ Each rule is stated in full on the page named beside it.
- **Run writes one at a time.** A database accepts one change at a
time, so deploy services one by one, each with `--wait`. See the
index's "Resource status".
- **Pass `--force` only when the user asked for an unattended run.**
Eleven verbs prompt, four of them not spelled `delete`. See the
index's "Destructive operations".
- **A request that names the action and its target is consent: pass
`--force`.** That consent covers the named action only, so ask
before any other destructive step it turns out to need. Eleven verbs
prompt, four of them not spelled `delete`. See the index's
"Destructive operations".
- **An empty list proves nothing about entitlement.** Some verbs are
plan-gated and some answer an empty array on any plan. See the
index's "Plan entitlement".
Expand Down
4 changes: 3 additions & 1 deletion plugins/pgedge/skills/pgedge-controlplane/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,9 @@ Each rule is stated in full on the page named beside it.
- **`--force` only skips the confirmation prompt.** A server-side
waiver is its own flag, `--force-unmodifiable`, or `--force-lost` on
`host remove`. See `database`, `database instance`, `database node`
and `host`.
and `host`. A request that names the action and its target is
consent for `--force`, and for that action only: ask before a
waiver flag or any other destructive step it turns out to need.
- **Start every database from `database init`.** When it can reach a
systemd Control Plane, it fills in the distinct ports each node
needs. The CLI refuses a spec that still carries a `CHANGE-ME`
Expand Down
6 changes: 4 additions & 2 deletions plugins/pgedge/skills/pgedge-managed/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,8 +116,10 @@ Each rule is stated in full on the page named beside it.
list` before you choose a value.
- **`--force` only skips the confirmation prompt.** It never deletes
branches: `--delete-branches` does that, and branch data cannot be
recovered. Pass `--force` only when the user asked for an unattended
run.
recovered. A request that names the action and its target is
consent, so pass `--force`. That consent covers the named action
only: ask before any other destructive step it turns out to need,
such as deleting branches.
- **Keep credentials out of the transcript.** Pass `--no-password` to
`database connection-string` whenever the output is shown to a
person. Otherwise write the output to a file or a variable, never to
Expand Down
Loading