Skip to content

fix: Change local dev port from 3001 to 3101#148

Merged
BenGWeeks merged 1 commit intomainfrom
fix/144-dev-port-3101
Feb 20, 2026
Merged

fix: Change local dev port from 3001 to 3101#148
BenGWeeks merged 1 commit intomainfrom
fix/144-dev-port-3101

Conversation

@EdiWeeks
Copy link
Contributor

@EdiWeeks EdiWeeks commented Feb 20, 2026

Summary

  • Changed the local dev server port from 3001 to 3101 in package.json
  • Updated CLAUDE.md to reflect the new port in redirect URI documentation
  • Updated Azure AD app registration to replace localhost:3001 with localhost:3101
image

Closes #144

Test plan

  • Dev server starts on port 3101
  • Azure AD redirect URI updated and verified

🤖 Generated with Claude Code

Closes #144

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the project’s local development setup to standardize the Next.js dev server port on 3101, and aligns Azure AD redirect URI documentation accordingly (per Issue #144).

Changes:

  • Updated bun run dev to start Next.js on port 3101.
  • Updated CLAUDE.md to document the new local dev redirect URI and the Azure CLI update command.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Changes the dev script to run next dev on port 3101.
CLAUDE.md Updates documented redirect URIs and the Azure AD CLI command to use localhost:3101.
Comments suppressed due to low confidence (2)

CLAUDE.md:136

  • The sample az ad app update --spa-redirect-uris ... command overwrites the entire redirect URI list, but this example omits the staging slot URI listed above. Someone copy/pasting it could accidentally remove existing redirect URIs (e.g., staging slots) from the app registration; update the example to include the full list or explicitly call out all URIs that must be preserved.
az ad app update --id "44c618b5-89c3-4673-92ec-f7afb4e403bf" --spa-redirect-uris "http://localhost:3000" "http://localhost:3101" "https://getthyme.ai"

CLAUDE.md:126

  • There are still repo docs referencing http://localhost:3001 (e.g., .claude/skills/capture-app-screenshots.md). With the dev port now set to 3101, those instructions will be incorrect; please update any remaining references so local setup/docs stay consistent.
- `http://localhost:3000` - Default Next.js port
- `http://localhost:3101` - Local dev port (configured in package.json)
- `https://getthyme.ai` - Production

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@EdiWeeks
Copy link
Contributor Author

Tested, reviewed by Copilot, @BenGWeeks check and merge please

@BenGWeeks BenGWeeks merged commit 7125176 into main Feb 20, 2026
7 checks passed
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.

[ENHANCEMENT] Run Thyme always on Port: 3101 locally

3 participants