Phase 1: Percent-encode the repository slug as one glab api path segment #561

Merged
claude merged 3 commits from IW-558-phase-01 into IW-558 2026-09-14 22:23:39 +00:00
Member

Phase 01 implementation for IW-558.

Files

Browse changed files: https://code.iterative.works/iterative-works/iw-cli/-/blob/IW-558-phase-01/

Phase 01 implementation for IW-558. ### Files Browse changed files: https://code.iterative.works/iterative-works/iw-cli/-/blob/IW-558-phase-01/
Percent-encode the repository slug as one glab api path segment
All checks were successful
CI / changes (pull_request) Successful in 2s
CI / contract (pull_request) Has been skipped
CI / format (pull_request) Successful in 7s
CI / lint (pull_request) Successful in 8s
CI / compile (pull_request) Successful in 1m17s
CI / test (pull_request) Successful in 14m5s
CI / dashboard-build (pull_request) Successful in 1m6s
508805d1fc
- encodedRepo now percent-encodes the whole slug, with '+' written as %20, so ? # & % and space can no longer end the path or add a parameter
- One central change fixes all four glab api path builders; no call site was edited
- Two regression tests, both proved RED first, one of them through buildMergeRequestForBranchCommand where the slug precedes a real query string
- Existing assertions pinning group%2Fsub%2Fproject pass unchanged: full encoding is byte-identical for a legal slug
chore(IW-558): update review-state for phase 01
All checks were successful
CI / contract (pull_request) Has been skipped
CI / compile (pull_request) Successful in 1m24s
CI / changes (pull_request) Successful in 2s
CI / format (pull_request) Successful in 8s
CI / lint (pull_request) Successful in 8s
CI / dashboard-build (pull_request) Successful in 1m7s
CI / test (pull_request) Successful in 14m50s
69ca4208e8
claude merged commit e3cc2b9cc6 into IW-558 2026-09-14 22:23:39 +00:00
claude deleted branch IW-558-phase-01 2026-09-14 22:23:39 +00:00
Sign in to join this conversation.
No description provided.