Skip to content

refactor(cmd): rename proflags import alias to platformflags - #1686

Merged
skevetter merged 1 commit into
mainfrom
refactor/platform-rename-proflags-alias
Oct 24, 2026
Merged

skevetter merged 1 commit into
mainfrom
refactor/platform-rename-proflags-alias

Conversation

@skevetter

@skevetter skevetter commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Rename the Go import alias proflags to platformflags for github.com/devsy-org/devsy/cmd/pro/flags (package name flags unchanged).
  • Token-only change across 20 files under cmd/pro; 56 lines changed (+56/-56).
  • No behaviour, flag, output, or import-path change.

Why

The alias proflags carries the old product name (Pro). Renaming it is a prerequisite for the cmd/pro -> cmd/platform directory 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; platformflags was unused before.
  • go build ./...: pass
  • GOOS=windows go vet ./cmd/pro/...: pass
  • mise run cli:format: no changes beyond the rename
  • mise run cli:lint:ci: 0 issues
  • mkdir -p dist && go test ./cmd/pro/...: pass
  • Pro-token PCRE count in cmd/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 --committed returned rate_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.go and login_test.go).

Remaining

  • proflags: 0.
  • cmd/pro Pro-token lines: 131 (PCRE above).

Next

cmd/pro -> cmd/platform directory move after cmd/root.go is free of open PRs #1581 and #1588.

Summary by CodeRabbit

  • Refactor
    • Updated the shared configuration used across platform commands. Existing command behavior and available options remain unchanged.
  • Tests
    • Updated command test setup to use the shared configuration; test coverage and assertions are unchanged.

@skevetter skevetter added the rename Loft/Pro rename label Oct 24, 2026
@netlify

netlify Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for devsydev canceled.

Name Link
🔨 Latest commit ff8645d
🔍 Latest deploy log https://app.netlify.com/projects/devsydev/deploys/6adc59123a7d959cddc4df4a

@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c910f2a8-106c-40f1-aaa4-4be71b99a6de
Commits

Reviewing files that changed from the base of the PR and between df78ff3 and ff8645d.

Files selected for processing: 20
  • cmd/pro/cluster/add.go
  • cmd/pro/cluster/add_flags.go
  • cmd/pro/cluster/cluster.go
  • cmd/pro/cmdutil/cmdutil.go
  • cmd/pro/cmdutil/cmdutil_test.go
  • cmd/pro/daemon/netcheck.go
  • cmd/pro/daemon/start.go
  • cmd/pro/daemon/status.go
  • 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/import.go
  • cmd/pro/workspace/workspace.go

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 0 remain after this review.

Walkthrough

The cmd/pro package now uses the platformflags alias for its flags package. Command fields, constructor signatures, host bindings, root setup, and test fixtures use the updated alias; behavior and test assertions are unchanged.

Changes: Platform flags alias update

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: Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Refactor

Possibly related issues

  • devsy-org/devsy#1551: The issue tracks this exact proflags to platformflags alias 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: Passed 5
Check name Status Explanation
Title check Passed The title clearly and concisely describes the alias rename across the command package.
Description check Passed 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 check…
Docstring Coverage Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 17 functions across 20 files.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@netlify

netlify Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for images-devsy-sh canceled.

Name Link
🔨 Latest commit ff8645d
🔍 Latest deploy log https://app.netlify.com/projects/images-devsy-sh/deploys/6adc591274ae43c1bbd82040

@skevetter
skevetter marked this pull request as ready for review October 24, 2026 08:11
@skevetter

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@mergify

mergify Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

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.

@skevetter

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@skevetter

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@skevetter

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@skevetter
skevetter merged commit b80762f into main Oct 24, 2026
114 checks passed
@skevetter
skevetter deleted the refactor/platform-rename-proflags-alias branch October 24, 2026 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rename Loft/Pro rename size/m

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant