add projects update command#165
Conversation
Supports renaming and archiving/reactivating projects via --name and --status flags. Accepts ID or name as the positional argument. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
Firetiger deploy monitoring skipped This PR didn't match the auto-monitor filter configured on your GitHub connection:
Reason: PR adds a CLI command for projects management, but does not modify API endpoints (packages/api/cmd/api/) or Temporal workflows (packages/api/lib/temporal) as specified in the filter. To monitor this PR anyway, reply with |
There was a problem hiding this comment.
lgtm, one minor thing is that if a user tries to enter an empty string for the update, we could check that locally on the CLI, rather than relying on the round-trip to the API.
non-blocking, because it functions the same regardless and create does the same thing
Summary
kernel projects update <id-or-name>command supporting--name(rename) and--status active|archived(archive/reactivate).resolveProjectArg.--nameor--status; rejects unknown status values.Test plan
make test— unit tests pass (3 new tests covering happy path, missing-flag, bad-status).projects create cli-test-update-flow→ success.projects update <id> --name cli-test-renamed→ renamed.projects update cli-test-renamed --status archived→ archived (resolution by name works).projects update <id> --status active --name cli-test-both→ both fields applied at once.projects update <id>(no flags) → error.projects update <id> --status deleted→ error listing valid values.projects delete <id>→ cleaned up.🤖 Generated with Claude Code
Note
Low Risk
Low risk CLI change that adds a new command and corresponding SDK call wiring; main risk is limited to user-facing validation and request parameter construction.
Overview
Adds a new
kernel projects update <id-or-name>command to rename projects and/or change their status toactiveorarchived, resolving the identifier via the existing ID/name lookup.Extends the
ProjectsServiceinterface and CLI handler to call the SDKUpdateendpoint, validates that at least one of--name/--statusis provided (and rejects unknown statuses), and prints the updated project details.Updates tests with a fake
Updateimplementation and adds coverage for the happy path, missing-flags error, and invalid-status error.Reviewed by Cursor Bugbot for commit 48698f0. Bugbot is set up for automated code reviews on this repo. Configure here.