Repository navigation
refactor(cmd): rename proflags import alias to platformflags - #1686
Conversation
✅ Deploy Preview for devsydev canceled.
|
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update cmd/pro flags references cmd/pro/cluster/*, cmd/pro/cmdutil/*, cmd/pro/daemon/*, cmd/pro/list.go, cmd/pro/login.go, cmd/pro/login_test.go, cmd/pro/logout.go, cmd/pro/pro.go, cmd/pro/project/project.go, cmd/pro/start.go, cmd/pro/start_flags.go, cmd/pro/template/template.go, cmd/pro/user/user.go, cmd/pro/workspace/* |
The flags package alias changes from proflags to platformflags. Command fields and constructor parameters, shared host helpers, environment bindings, root setup, and test fixtures use the updated alias. Command behavior and test assertions remain unchanged. |
Priority:
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Refactor
Possibly related issues
devsy-org/devsy#1551: The issue tracks this exactproflagstoplatformflagsalias change in cmd/pro and lists this pull request as the corresponding item.
Suggested labels: surface-refactor
Merge Risk: 
Minimal · up to ff864
-
No concrete merge-blocking risk was found; the alias rename preserves existing behavior and contracts.
Pre-merge checks: 
5
-
Full details: Description check
-
Explanation
The description explains what changed, why it changed, the refactor scope, validation commands, and expected behavior. It does not use all template sections or provide explicit AI-assistance and checklist confirmations, but it is mostly complete for this alias-only refactor.
-
Finishing Touches
-
✨ Simplify code
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
✅ Deploy Preview for images-devsy-sh canceled.
|
|
@coderabbitai review |
|
|
This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again. |
|
@coderabbitai review |
|
|
@coderabbitai review |
|
|
@coderabbitai review |
✅ Action performed
|
Summary
proflagstoplatformflagsforgithub.com/devsy-org/devsy/cmd/pro/flags(package nameflagsunchanged).cmd/pro; 56 lines changed (+56/-56).Why
The alias
proflagscarries the old product name (Pro). Renaming it is a prerequisite for thecmd/pro->cmd/platformdirectory move, which comes in a later PR.Tier
T1: internal Go import alias, all callers in this repo, behaviour-preserving.
Validation
git grep -n proflags: 56 lines before, 0 after;platformflagswas unused before.go build ./...: passGOOS=windows go vet ./cmd/pro/...: passmise run cli:format: no changes beyond the renamemise run cli:lint:ci: 0 issuesmkdir -p dist && go test ./cmd/pro/...: passcmd/pro(\bpro\b|Pro(?=[A-Z0-9_]|\b)|(^|[^A-Za-z])pro(?=[A-Z])|DEVSY_PRO(_|$)): 131 before, 131 after (the alias is not a Pro-token match).CodeRabbit CLI
Not completed:
coderabbit review --agent --committedreturnedrate_limit("Rate limit exceeded", wait about 2m10s, free OSS quota exhausted). GitHub CodeRabbit review still gates.Auto-merge
Eligible: yes. T1, 20 files (cap 25), 56 added and 56 removed lines total (48 and 48 non-test; the rest are
cmdutil_test.goandlogin_test.go).Remaining
proflags: 0.cmd/proPro-token lines: 131 (PCRE above).Next
cmd/pro->cmd/platformdirectory move aftercmd/root.gois free of open PRs #1581 and #1588.Summary by CodeRabbit