Wrap the variable, application-delete and isolation-create verbs #11

Open

geusebio wants to merge 1 commit from feature/terraform-variable-wrappers into main

pull from: feature/terraform-variable-wrappers

merge into: Grey.ooo:main

Grey.ooo:main

Grey.ooo:feature/deployment-progress-endpoints-frame

Grey.ooo:feature/failure-reference-on-the-wire

Grey.ooo:feature/conflict-details-on-an-api-error

Grey.ooo:feature/revision-lookup-and-status-reason

Grey.ooo:feature/managed-service-verbs

Grey.ooo:feature/browser-session

Grey.ooo:feature/static-analysis-and-coverage-gate

Grey.ooo:feature/credit-read

Grey.ooo:feature/identity-read

Grey.ooo:feature/deployment-state-and-toggles

Conversation 0 Commits 1 Files changed 6 +528 -6

geusebio commented 2026-08-19 20:54:31 +00:00

Owner

Copy link

Adds the /cli wrappers the new Terraform provider needs, keeping one
definition of each verb on the shared contract (the reason this module exists).

What

• Variables / SetVariables / DeleteVariable — the application
  configuration layer (GET/PUT/DELETE /cli/applications/{slug}/variables).
  Values are never on the wire, so Variable carries metadata only.
• DeleteApplication — DELETE /cli/applications/{slug}, with ?force for a
  still-bound managed service. Paired with the platform's new endpoint (separate PR).
• CreateApplicationWithIsolation — a superset of CreateApplication naming
  the isolation level. The 3-arg form delegates and omits the key when unset, so
  the platform still applies its default and the agent's one caller is untouched.

Verification

• go test ./... green; hack/lint.sh reports 0 issues.
• Coverage ratchet 88.8 → 90.7% (floor bumped in this commit).
• Stdlib only — no new dependencies.

Adds the `/cli` wrappers the new Terraform provider needs, keeping one definition of each verb on the shared contract (the reason this module exists). ## What - **`Variables` / `SetVariables` / `DeleteVariable`** — the application configuration layer (`GET`/`PUT`/`DELETE /cli/applications/{slug}/variables`). Values are never on the wire, so `Variable` carries metadata only. - **`DeleteApplication`** — `DELETE /cli/applications/{slug}`, with `?force` for a still-bound managed service. Paired with the platform's new endpoint (separate PR). - **`CreateApplicationWithIsolation`** — a superset of `CreateApplication` naming the isolation level. The 3-arg form delegates and omits the key when unset, so the platform still applies its default and the agent's one caller is untouched. ## Verification - `go test ./...` green; `hack/lint.sh` reports 0 issues. - Coverage ratchet 88.8 → **90.7%** (floor bumped in this commit). - Stdlib only — no new dependencies.

geusebio added 1 commit 2026-08-19 20:54:31 +00:00

Wrap the variable, application-delete and isolation-create verbs ... 30e9065074

Adds the wrappers a declarative consumer (the new Terraform provider) needs and
the CLI did not yet require, keeping one definition of each verb on the contract:

- Variables/SetVariables/DeleteVariable — the application configuration layer
  (GET/PUT/DELETE /cli/applications/{slug}/variables). Values are never on the
  wire, so Variable carries metadata only.
- DeleteApplication — DELETE /cli/applications/{slug}, with ?force for a still-
  bound managed service. Paired with the platform's new endpoint.
- CreateApplicationWithIsolation — a superset of CreateApplication that names the
  isolation level; the 3-arg form delegates to it and omits the key when unset,
  so the platform still reads its default and the agent's caller is untouched.

Stdlib only, matching the module's no-dependency rule. Coverage floor 88.8 -> 90.7.

All checks were successful

Hide checks

CI / build (pull_request) Successful in 1m24s

Details

This pull request has changes conflicting with the target branch.

• README.md
• client/cliapi_test.go
• coverage.floor

View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Merge commit title Wrap the variable, application-delete and isolation-create verbs (#11)

Merge commit body Reviewed-on: https://git.grey.ooo/Grey.ooo/someones.computer_scclient/pulls/11

Wrap the variable, application-delete and isolation-create verbs (#11) Reviewed-on: https://git.grey.ooo/Grey.ooo/someones.computer_scclient/pulls/11 Copy merge commit message

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin feature/terraform-variable-wrappers:feature/terraform-variable-wrappers

git switch feature/terraform-variable-wrappers

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main

git merge --no-ff feature/terraform-variable-wrappers

git switch feature/terraform-variable-wrappers

git rebase main

git switch main

git merge --ff-only feature/terraform-variable-wrappers

git switch feature/terraform-variable-wrappers

git rebase main

git switch main

git merge --no-ff feature/terraform-variable-wrappers

git switch main

git merge --squash feature/terraform-variable-wrappers

git switch main

git merge --ff-only feature/terraform-variable-wrappers

git switch main

git merge feature/terraform-variable-wrappers

git push origin main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

Grey.ooo/someones.computer_scclient!11

No description provided.