feat: sync CLI with dashboard@v1.6.16 - #85
Open
cursor[bot] wants to merge 1 commit into
Open
Conversation
Sync CLI with dashboard@v1.6.16 (dashboard#260). The 1.6.15 -> 1.6.16 bundled .d.ts is bit-identical: dashboard#260 only adds server-side logging when a phone OTP resend follows three failed verifications on the real POST /api/otp/phone2/send path. No route or DTO change reaches the CLI. Since main was still pinned to 1.6.12, this bump also picks up the accumulated additive changes from v1.6.14/v1.6.15: - GET /api/projects/:id gained avatarUrl / plan / platforms / userCount - POST /api/otp/phone2/send (real phone-OTP send path) was added alongside the legacy POST /api/otp/phone/send decoy Surface plan, platforms, and userCount in `photon projects show`. Skip avatarUrl (redundant with `spectrum avatar`, CLI can't render images). The OTP routes are auth-flow only and the CLI does not call them. Co-authored-by: citron <lcandy2@users.noreply.github.com>
Agent PR ReviewScenario: A — Initial review (PR opened by cursor[bot]) What's right
Concerns
Deferred (acknowledged in PR body)
Final noteClean, minimal upstream-sync PR. Marked ready for review and labeled |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upstream version
cdcd44b2a4012d819ed3e9a83c54973fa9f83b8d)@photon-ai/dashboard-api1.6.12 → 1.6.16Routes added/removed/changed
Spans dashboard releases v1.6.13 → v1.6.16. See
UPSTREAM_DIFF.md.POST /api/otp/phone2/send— real phone-OTP send path introduced in dashboard#259 (v1.6.15). The legacyPOST /api/otp/phone/sendroute is preserved as a decoy honeypot. CLI never callsapi.otp.*, so no command change is needed.GET /api/projects/:id— response gainedavatarUrl: string | null,plan: PlanTier,platforms: SpectrumPlatformId[],userCount: number(dashboard#255, v1.6.14). Surfacedplan,platforms, anduserCountinphoton projects show.avatarUrlis skipped — CLI can't render images and it's redundant with thespectrum avatarcommands.POST /api/otp/phone/send— cosmeticcode/captchaTokenbody-field reordering (v1.6.15). No behavior change; CLI does not call this route.>3-dotssignup suspension) and v1.6.16 (dashboard#260, log real phone sends after three OTP failures) are pure server-side changes with bit-identical@photon-ai/dashboard-apidist output.Snapshot changes
New runtime dependencies
@photon-ai/dashboard-apidevDependency was bumped.Checklist
@photon-ai/dashboard-apiversion bump, not hand editsbun run checkpasses locallyNeed help on this PR? Tag
/codesmithwith what you need. Autofix is disabled.Note
Low Risk
Low-risk dependency bump plus read-only display fields on an existing show command; no auth, billing, or OTP behavior changes in the CLI.
Overview
Syncs the CLI with dashboard
@photon-ai/dashboard-api1.6.12 → 1.6.16 and refreshesUPSTREAM_DIFF.mdfor the v1.6.13–v1.6.16 route delta (notablyPOST api.otp.phone2.sendand richerGET api.projects.:idresponses).photon projects shownow printsplan,platforms, anduserCountfrom the updated project detail payload;avatarUrlis intentionally omitted from the terminal view.No new runtime dependencies and no CLI commands added for OTP routes—the CLI does not call
api.otp.*.Reviewed by Cursor Bugbot for commit 730ecea. Bugbot is set up for automated code reviews on this repo. Configure here.