Skip to content

Improve developer experience baseline#108

Open
deancochran wants to merge 1 commit into
devfrom
dx/developer-experience-baseline
Open

Improve developer experience baseline#108
deancochran wants to merge 1 commit into
devfrom
dx/developer-experience-baseline

Conversation

@deancochran

Copy link
Copy Markdown
Owner

Summary

  • Add a repo doctor script for local tool/env/port diagnostics.
  • Split web dev startup into local and tunnel modes so default web dev no longer requires ngrok.
  • Add validation aliases, CI validation workflow, and checked-in env templates.
  • Refresh README guidance for the current shared dev workflow and existing mobile scripts.

Validation

  • pnpm install --frozen-lockfile
  • pnpm run doctor
  • pnpm check
  • pnpm check-types
  • pnpm test
  • pnpm knip

Notes

  • pnpm run doctor emits warnings for optional local tools/env files when they are not installed or copied locally.

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c4ea8ef4d4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/web/.env.example
AUTH_EMAIL_FROM=GradientPeak Local <noreply@localhost>
AUTH_EMAIL_REPLY_TO=noreply@localhost
AUTH_SMTP_HOST=127.0.0.1
AUTH_SMTP_PORT=54324

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Point local SMTP at Inbucket's SMTP port

With this template, any developer who copies apps/web/.env.example and uses AUTH_EMAIL_MODE=smtp will send Nodemailer to port 54324. In our Supabase config that port is the Inbucket web UI (packages/db/supabase/config.toml:92-95); the SMTP listener is 54325, so verification/reset emails fail to send instead of appearing in Inbucket. Set this to 54325 or change the Supabase SMTP port to match.

Useful? React with 👍 / 👎.

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