Skip to content

feat(setup): add incremental Chat setup - #7043

Merged
TheodoreSpeaks merged 2 commits into
stagingfrom
feat/sim-setup-chat
Aug 24, 2026
Merged

feat(setup): add incremental Chat setup#7043
TheodoreSpeaks merged 2 commits into
stagingfrom
feat/sim-setup-chat

Conversation

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator

Summary

  • add sim-setup add chat for configuring Chat without rerunning the wizard
  • preserve configuration when no Chat API key is received
  • bump sim-setup to 1.0.2

Type of Change

  • New feature

Testing

  • bun run lint
  • bun run check:audits
  • bun run --cwd packages/sim-setup test
  • bun run --cwd packages/sim-setup type-check
  • bun run --cwd packages/sim-setup build

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@TheodoreSpeaks
TheodoreSpeaks requested a review from a team as a code owner August 24, 2026 19:53
@vercel

vercel Bot commented Aug 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Aug 24, 2026 8:06pm

Request Review

@cursor

cursor Bot commented Aug 24, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Incremental CLI configuration with tests; no changes to runtime auth or app request handling beyond env key registration for setup tooling.

Overview
Adds npx sim-setup add chat so operators can set or replace the Chat API key without rerunning the full wizard, alongside updates to CLI help and README examples.

The chat path prompts for COPILOT_API_KEY, applies SIM_AGENT_API_URL (mothership override) and NEXT_PUBLIC_CHAT_DISABLED via existing wizard helpers, and writes through reconcileEnvValues to the detected install (e.g. Compose .env). If no key is provided, setup throws and leaves configuration unchanged.

Chat is registered as a setup feature but omitted from sim-setup config capability ordering (same pattern as OAuth integrations). DEPLOYMENT_CONFIGURATION_KEYS now includes the chat-related env vars so discovery and shell-shadowing logic treat them as managed deployment config. sim-setup is bumped to 1.0.2.

Reviewed by Cursor Bugbot for commit 85db8e5. Bugbot is set up for automated code reviews on this repo. Configure here.

@greptile-apps

greptile-apps Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds an incremental command for configuring Chat without rerunning the complete setup wizard and updates configuration discovery to retain existing Chat values.

  • Adds sim-setup add chat command routing, prompting, and focused tests.
  • Includes Chat environment keys in deployment configuration discovery.
  • Updates package metadata and usage documentation for version 1.0.2.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
packages/sim-setup/src/feature-setup.ts Adds the incremental Chat setup branch, preserving configuration when no API key is received.
packages/deployment-config/src/env-capabilities.ts Adds Chat-related environment variables to deployment configuration discovery.
packages/sim-setup/src/feature-setup-chat.test.ts Tests focused Chat configuration writes and the no-key cancellation path.
packages/sim-setup/src/capability-config.ts Registers Chat as an available incremental setup feature.
packages/sim-setup/src/setup-status.ts Keeps Chat outside the capability-status ordering after adding it to the setup feature catalog.

Reviews (3): Last reviewed commit: "fix(setup): retain existing Compose Chat..." | Re-trigger Greptile

Comment thread README.md
@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@greptile

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@cursor review

Comment thread packages/sim-setup/src/feature-setup.ts
@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@greptile

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 85db8e5. Configure here.

@TheodoreSpeaks
TheodoreSpeaks merged commit 7b761ba into staging Aug 24, 2026
30 checks passed
@TheodoreSpeaks
TheodoreSpeaks deleted the feat/sim-setup-chat branch August 24, 2026 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant