-
Notifications
You must be signed in to change notification settings - Fork 6
feat: Enhanced setup script with quick deploy and Azure infrastructur… #170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
feat: Enhanced setup script with quick deploy and Azure infrastructur… #170
Conversation
anfibiacreativa
commented
Oct 23, 2025
- Universal default handling (Enter/1/2/y/n support for all prompts)
- Quick deploy mode with --deploy flag for fast redeployment
- Smart production OAuth detection (skips wizard if already configured)
- Fixed Azure diagnostic settings (added workspaceId, removed deprecated retention policies)
- Optimized deployment workflow (azd provision + deploy.sh instead of azd up)
- Fixed azd environment name detection
- Removed duplicate dev setup prompts
- Version bump to 2.2.0
- Improved error messages and validation
- Configuration summary with masked secrets
…e fixes - Universal default handling (Enter/1/2/y/n support for all prompts) - Quick deploy mode with --deploy flag for fast redeployment - Smart production OAuth detection (skips wizard if already configured) - Fixed Azure diagnostic settings (added workspaceId, removed deprecated retention policies) - Optimized deployment workflow (azd provision + deploy.sh instead of azd up) - Fixed azd environment name detection - Removed duplicate dev setup prompts - Version bump to 2.2.0 - Improved error messages and validation - Configuration summary with masked secrets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces a quick deploy mode with the --deploy flag for fast redeployment, enhances the setup script with improved default handling and user prompts, adds support for separate production OAuth credentials, and fixes Azure diagnostic settings by adding the required workspaceId parameter and removing deprecated retention policies. Version bumped to 2.2.0.
Key Changes
- Added
--deployflag for quick redeployment using existing.envconfiguration - Introduced separate production OAuth credentials (
GITHUB_CLIENT_ID_PROD,GITHUB_CLIENT_SECRET_PROD) with automatic fallback to dev credentials - Fixed Azure diagnostic settings by adding
workspaceIdand removing deprecated retention policies
Reviewed Changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/full-setup.sh | Added quick deploy mode, version check, configuration validation/summary, enhanced prompts with universal default handling, and production OAuth setup |
| packages/server/src/shared/env.ts | Added logic to select production OAuth credentials based on environment |
| package.json | Bumped version from 1.0.0 to 2.2.0 |
| infra/main.bicep | Added logAnalyticsWorkspaceId parameter to Cosmos DB module and dependency on container apps environment |
| infra/database.bicep | Added logAnalyticsWorkspaceId parameter and removed deprecated retention policies from diagnostic settings |
| .env.example | Added production OAuth credential placeholders and documentation |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
- Added Quick Redeploy Mode section to README - Documented --deploy flag for fast redeployment - Updated Azure deployment steps (azd provision + deploy.sh) - Added troubleshooting for timeout and stuck deployments - Documented universal input handling (Enter/1/2/y/n) - Updated all deployment guides with new workflow
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the setup script is really smooth 🙌