Deferred, non-blocking follow-ups from the PR #151 review (normalize MCP registration envelopes → {success, <entity>, message?}). Coordinated breaking change with awsbnkctl/internal/forge/client.go.
1. Envelope sweep — update_project, delete_project, update_cluster, delete_cluster (and likely others) still have inconsistent shapes. Sweep them to the same {success, <entity>, message?} envelope for a consistent MCP contract.
2. Edge-branch tests — add tests for the explicit success: False path and the non-dict {success: False, raw} fallback in create_project/create_cluster.
3. Harden success derivation — create_cluster hardcodes success: True; derive from status != "failed" if exposed. Tighten error passthrough to key on ok is False alone.
Source: PR #151 review (mwiget).
Migrated from sp-prod-field/bnk-forge #156 (opened 2026-05-26; original labels: enhancement). That repository is archived and read-only.
Bare #NNN references in the text above refer to issues and PRs in the original repository, not to numbering here.
Deferred, non-blocking follow-ups from the PR #151 review (normalize MCP registration envelopes →
{success, <entity>, message?}). Coordinated breaking change withawsbnkctl/internal/forge/client.go.1. Envelope sweep —
update_project,delete_project,update_cluster,delete_cluster(and likely others) still have inconsistent shapes. Sweep them to the same{success, <entity>, message?}envelope for a consistent MCP contract.2. Edge-branch tests — add tests for the explicit
success: Falsepath and the non-dict{success: False, raw}fallback increate_project/create_cluster.3. Harden success derivation —
create_clusterhardcodessuccess: True; derive fromstatus != "failed"if exposed. Tighten error passthrough to key onok is Falsealone.Source: PR #151 review (mwiget).
Migrated from
sp-prod-field/bnk-forge#156 (opened 2026-05-26; original labels:enhancement). That repository is archived and read-only.Bare
#NNNreferences in the text above refer to issues and PRs in the original repository, not to numbering here.