Skip to content

disable example app#6

Merged
remcostoeten merged 1 commit into
masterfrom
build
Sep 19, 2025
Merged

disable example app#6
remcostoeten merged 1 commit into
masterfrom
build

Conversation

@remcostoeten

@remcostoeten remcostoeten commented Sep 19, 2025

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Removed a third-party environment validation dependency to reduce footprint.
    • Cleaned up environment configuration no longer required by the app.
    • Removed the database seeding script to streamline the example app setup.
  • Refactor
    • Eliminated legacy setup and utilities tied to the removed dependency and seeding workflow.
  • Tests
    • No changes.
  • Documentation
    • No user-facing documentation changes.

@remcostoeten remcostoeten merged commit dbac561 into master Sep 19, 2025
1 check failed
@coderabbitai

coderabbitai Bot commented Sep 19, 2025

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

Removed the env validation dependency from package.json and deleted two server files: the environment configuration module and the database seeding script. This eliminates the exported env constant and the exported seedDatabase function previously used for seeding via Drizzle ORM.

Changes

Cohort / File(s) Summary
Dependency removal
apps/example-app/package.json
Removed dependency @t3-oss/env-nextjs (^0.13.8). No other changes.
Server env config removal
apps/example-app/src/server/env.ts
Deleted file that created and exported env via createEnv with DATABASE_URL validation using zod.
Seeding script removal
apps/example-app/src/server/seed.ts
Deleted database seeding script using Drizzle ORM and env.DATABASE_URL; removed exported seedDatabase() and CLI execution path.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Dev as Developer
  participant CLI as seed.ts (deleted)
  participant ORM as Drizzle ORM
  participant DB as Database
  participant Env as process.env

  Note over CLI,DB: Previous seeding flow (removed)
  Dev->>CLI: Run seed script
  CLI->>Env: Read DATABASE_URL
  CLI->>ORM: Initialize client with DATABASE_URL
  ORM->>DB: Connect and execute deletes/inserts
  DB-->>ORM: Ack results
  ORM-->>CLI: Operation results
  CLI-->>Dev: Log success summary
  alt Error
    CLI-->>Dev: Log error
    CLI->>CLI: process.exit(1)
  end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

Thump-thump, I tidy the burrow’s code,
Two carrots gone from the seeding road.
Env winds quiet, no schema breeze—
I twitch my nose at simpler trees.
Fewer hops, a lighter load—
Onward! down this streamlined node. 🥕✨


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Free

📥 Commits

Reviewing files that changed from the base of the PR and between 4c92a13 and a45ec45.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (3)
  • apps/example-app/package.json (0 hunks)
  • apps/example-app/src/server/env.ts (0 hunks)
  • apps/example-app/src/server/seed.ts (0 hunks)

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

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

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