Skip to content

enhance: Codex向けLAN HTTPS開発環境を追加 - #25

Open
mattyatea wants to merge 2 commits into
developfrom
codex/add-codex-dev-environment
Open

enhance: Codex向けLAN HTTPS開発環境を追加#25
mattyatea wants to merge 2 commits into
developfrom
codex/add-codex-dev-environment

Conversation

@mattyatea

Copy link
Copy Markdown

What

  • Codex Local Environment のセットアップと Run アクションを追加
  • worktree ごとに固有の PostgreSQL データベースと Portless 状態・公開ポートを生成
  • 共有 Redis の DB 1 を利用
  • Portless 経由の LAN HTTPS URL と pnpm dev の HMR を有効化

Why

複数の Codex Environment を同時に立ち上げてもデータベースや Portless が競合せず、LAN 内の別端末から HTTPS で開発中の Misskey を確認できるようにするためです。

Additional info (optional)

初回のみ Prepare LAN HTTPS アクションで mDNS ツールの導入とローカル CA の信頼設定を行います。PostgreSQL と Redis は 127.0.0.1:5432 / 127.0.0.1:6379 の既存サービスを利用します。

動作確認:

  • Codex Environment TOML の構文確認
  • 固有 PostgreSQL DB の作成・migration
  • Redis DB 1 の接続
  • LAN HTTPS URL の発行
  • Vite HMR の wss 接続
  • 既存 Portless プロキシとの分離

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • Check that there are no unintended changes
  • Confirmed that there are no security issues
  • Add or update tests where applicable
  • Add or update documentation where applicable
  • Add changes to the CHANGELOG.md (not applicable: development environment only)

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@mattyatea, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 34 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: a2817918-795a-48a5-87a5-ec530815bffa

📥 Commits

Reviewing files that changed from the base of the PR and between 97a7973 and ac2d9ad.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (3)
  • .codex/environments/environment.toml
  • package.json
  • scripts/setup-codex-environment.mjs

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@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: ac2d9ad18d

ℹ️ 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 on lines +140 to +141
const config = yaml.load(await fsp.readFile(sourceConfigPath, 'utf-8')) ?? {};
config.url = 'https://misskey.local/';

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Require a setup password before publishing to the LAN

On a fresh worktree/database, this copies .config/example.yml, where setupPassword is commented out, and the Run action subsequently exposes the instance through Portless LAN mode. In that state, admin/accounts/create explicitly accepts an unauthenticated request as the first administrator when config.setupPassword is unset, so another device on the LAN can claim the new instance before the developer does. Generate a per-environment setup password in the ignored config and present it to the developer before allowing the LAN server to start.

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