Skip to content

[codex] converge app on Sealos codex flow#6

Merged
Che-Zhu merged 1 commit intomainfrom
codex/codex-devbox-gateway-convergence
Apr 24, 2026
Merged

[codex] converge app on Sealos codex flow#6
Che-Zhu merged 1 commit intomainfrom
codex/codex-devbox-gateway-convergence

Conversation

@Che-Zhu
Copy link
Copy Markdown
Collaborator

@Che-Zhu Che-Zhu commented Apr 24, 2026

What changed

This change converges the app around a Sealos-specific Codex execution flow instead of the older multi-agent, multi-entry UX.

It removes legacy home page and multi-repository UI paths, simplifies repository and task surfaces, updates the database schema and migration set for the new flow, and rewrites the project documentation to match the current product direction.

Why it changed

The repository had drifted between an older generic coding-agent template and the newer Sealos-focused product direction.

This PR makes the product surface, schema, and documentation line up with the current execution model: a fixed codex + gpt-5.4 path running through Devbox and the Codex Gateway.

Developer impact

  • The old generic home-page and multi-repo UI code paths are removed.
  • The app now reads more clearly as a Sealos deployment workspace.
  • README is now developer-facing, with deeper material moved into reference/.
  • A new migration is included alongside the schema update.

Validation

  • pnpm format
  • pnpm type-check
  • pnpm lint (warnings only, no errors)

@Che-Zhu Che-Zhu marked this pull request as ready for review April 24, 2026 03:14
@Che-Zhu Che-Zhu merged commit f0a58c0 into main Apr 24, 2026
5 checks passed
@Che-Zhu Che-Zhu deleted the codex/codex-devbox-gateway-convergence branch April 24, 2026 03:17
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

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: 81619c6e2a

ℹ️ 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 drizzle.config.ts
Comment on lines +9 to +12
if (existsSync(envLocalPath)) {
config({ path: envLocalPath })
} else if (existsSync(envPath)) {
config({ path: envPath })
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 Load .env when .env.local is present but incomplete

The new dotenv loading logic only reads .env when .env.local does not exist, so Drizzle commands now fail if a repo has a partial .env.local and keeps POSTGRES_URL in .env (a common local setup). In that case POSTGRES_URL remains unset and the hard error is thrown even though a valid fallback file exists. Load both files in precedence order (.env.local first, then .env for missing keys) to preserve the documented fallback behavior.

Useful? React with 👍 / 👎.

Comment on lines +202 to +206
"idx": 28,
"version": "7",
"when": 1776910000000,
"tag": "0028_solid_lockjaw",
"breakpoints": true
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 Add the 0028 Drizzle snapshot metadata

This commit registers migration 0028_solid_lockjaw in _journal.json, but there is no corresponding lib/db/migrations/meta/0028_snapshot.json (snapshots currently stop at 0027_snapshot.json). Without that snapshot, future drizzle-kit generate runs diff against stale metadata and can repeatedly re-generate the same default-change migration, causing migration drift and noisy duplicate SQL.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant