mock converted from js to ts

[mertkaradayi]

Summary by CodeRabbit

• Refactor

  • Modernized module imports and streamlined environment variable loading for improved reliability and consistency.

• Chores

  • Added a script to generate mock data, simplifying local setup and testing.

[coderabbitai]

Walkthrough

Migrates protocol/generate-mock-data.ts from CommonJS to ES module imports, switches dotenv loading to import-time via dotenv/config, and adjusts db import to a default ESM import. Adds an npm script generate-mock in protocol/package.json to run the script with ts-node/register. No control-flow changes.

Changes

Cohort / File(s)	Summary of changes
ESM migration and env loading protocol/generate-mock-data.ts	Replaced require with ES module imports; loaded env via import 'dotenv/config'; switched db to default ESM import; retained existing functional logic and flow.
NPM scripts protocol/package.json	Added generate-mock script using ts-node/register to run the mock data generator; adjusted JSON formatting to include the new script.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I twitch my ears at modules new,
From require’s burrow, imports grew—
Dotenv breezes load the air,
A mock-carrot feast prepared with care.
One hop: npm run generate-mock,
Thump-thump—data sprouts around the clock! 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title Check	❓ Inconclusive	The title indicates a conversion from JavaScript to TypeScript but is too vague and poorly phrased because “mock” does not clearly identify the specific script or component being converted and it omits key context such as the generate-mock-data generator.	Please clarify and rephrase the title to specify the exact change, for example “Convert generate-mock-data script from JavaScript to TypeScript,” so that the primary change is immediately understandable.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/generate-mock-to-ts

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

protocol/package.json (1)

15-16: Script addition LGTM; consider preloading dotenv via CLI instead of side-effect import.

Current setup works. If you prefer keeping code free of side-effect imports, you can preload dotenv in the script and drop import 'dotenv/config' from the TS file.

Apply this to package.json:

-    "generate-mock": "TS_NODE_TRANSPILE_ONLY=1 node -r ts-node/register ./generate-mock-data.ts"
+    "generate-mock": "TS_NODE_TRANSPILE_ONLY=1 node -r dotenv/config -r ts-node/register ./generate-mock-data.ts"

Then remove the side-effect import at the top of generate-mock-data.ts:

-import 'dotenv/config';

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8e83605 and efa642b.

📒 Files selected for processing (2)

• protocol/generate-mock-data.ts (1 hunks)
• protocol/package.json (1 hunks)

🔇 Additional comments (2)

protocol/generate-mock-data.ts (2)

2-2: If keeping side-effect env load, it's fine as-is.

Using import 'dotenv/config' here is acceptable for a CLI script and loads envs early enough for downstream imports.

---

2-8: ESM import migration verified
tsconfig.json enables esModuleInterop & allowSyntheticDefaultImports; protocol/src/lib/db exports a default and ts-node/register under Node16 resolution will load it correctly.