[Fix] freestyle-mock honors $PORT, drop server.listen string-patch

[BilalG1]

Summary

The multi-worker freestyle-mock rewrite (#1430) hardcoded server.listen(8080), which collides with qstash inside the local-emulator container. Supervisord sets PORT=8180 for freestyle-mock specifically to avoid this clash, but the new source ignores process.env.PORT.

The local-emulator Dockerfile previously bridged this with a server.replace('server.listen(8080)', ...) string-patch on the embedded source. The new code is server.listen(8080, () => { ... }) — the literal 'server.listen(8080)' substring no longer matches, so the replace silently no-ops and freestyle-mock binds 8080. qstash then can't start (address already in use: 127.0.0.1:8080 → FATAL), the backend (which depends on qstash) never comes up, and the emulator smoke test times out.

Observed in this run:

smoke-test: FTL address already in use: 127.0.0.1:8080
smoke-test: WARN exited: qstash (exit status 1; not expected)
smoke-test: INFO gave up: qstash entered FATAL state, too many start retries too quickly
[603s] SMOKE TEST FAILED: backend /health?db=1 did not return 200 within 300s

Changes

• docker/dependencies/freestyle-mock/Dockerfile: server.listen(PORT) where PORT = process.env.PORT || 8080, plus the startup log reflects the actual port.
• docker/local-emulator/Dockerfile: drop the now-redundant string-replace for the listen call. The two remaining replaces (fs/promises import + node_modules symlink) are unrelated and kept.

Test plan

• QEMU emulator build workflow passes on this branch (smoke test reaches healthy backend).
• Verify locally that supervisord's PORT=8180 is honored by freestyle-mock and qstash binds 8080 cleanly.

Summary by CodeRabbit

• Chores
  • Server listening port is now configurable via PORT (default 8080).
  • Local emulator startup improved to better handle dependencies and create a node_modules symlink to streamline local development.
  • Seed/process timeout increased to 90s for reliability; local database statement timeout raised to 300s to accommodate longer queries.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
stack-auth-hosted-components	Ready	Preview, Comment	May 14, 2026 2:42am
stack-auth-mcp	Ready	Preview, Comment	May 14, 2026 2:42am
stack-backend	Ready	Preview, Comment	May 14, 2026 2:42am
stack-dashboard	Ready	Preview, Comment	May 14, 2026 2:42am
stack-demo	Ready	Preview, Comment	May 14, 2026 2:42am
stack-docs	Ready	Preview, Comment	May 14, 2026 2:42am
stack-preview-backend	Ready	Preview, Comment	May 14, 2026 2:42am
stack-preview-dashboard	Ready	Preview, Comment	May 14, 2026 2:42am

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c4f12c56-e54d-48ca-ab50-3d93203a0068

📥 Commits

Reviewing files that changed from the base of the PR and between 358df4e and b358370.

📒 Files selected for processing (1)

• docker/local-emulator/supervisord.conf

✅ Files skipped from review due to trivial changes (1)

• docker/local-emulator/supervisord.conf

---

📝 Walkthrough

Walkthrough

The PR makes the freestyle-mock server's listening port configurable via process.env.PORT (default 8080), patches the local-emulator extraction to symlink bundled node_modules and inject symlinkSync, raises Postgres statement_timeout to 300s, and increases a Prisma transaction timeout to 90_000 ms.

Changes

Freestyle-mock port configuration and emulator setup

Layer / File(s)	Summary
Configurable server port and emulator node_modules setup docker/dependencies/freestyle-mock/Dockerfile, docker/local-emulator/Dockerfile, docker/local-emulator/supervisord.conf	The freestyle-mock server now reads process.env.PORT (default 8080) and uses it for binding. The local emulator's extraction script patches the generated server.mjs to use that port, inject symlinkSync import from fs, and create a symlink from the bundled /app/freestyle-mock/node_modules into the emulator workDir (errors ignored). The supervisord Postgres command sets statement_timeout=300s.

Backend seeding timeout adjustment

Layer / File(s)	Summary
Extend Prisma transaction timeout apps/backend/src/lib/seed-dummy-data.ts	The Prisma globalPrismaClient.$transaction timeout used in seedDummySessionActivityEvents was increased from 30_000 ms to 90_000 ms.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• hexclave/stack-auth#1430: Both PRs modify the freestyle-mock’s generated server.mjs/Docker setup and address server binding behavior.

Poem

🐰 I hopped in code to change a port,
I taught the server to heed env's report.
A symlink ties the modules near,
The DB waits longer — seeds appear.
Logs whisper ports, the build is clear.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: making freestyle-mock honor the PORT environment variable and removing the string-patch approach.
Description check	✅ Passed	The description is comprehensive, providing context, root cause analysis, solution details, and a test plan. It follows the repository template structure and includes all necessary information for understanding the changes.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/freestyle-mock-port-env

---

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.}

[greptile-apps]

Greptile Summary

This fix makes the freestyle-mock server honour process.env.PORT (falling back to 8080), resolving a port collision between freestyle-mock and qstash inside the local-emulator container. The companion change removes the now-broken string-patch in the local-emulator Dockerfile that was silently no-oping after the server source changed its listen call signature.

• freestyle-mock/Dockerfile: Introduces const PORT = process.env.PORT || 8080 so supervisord's PORT=8180 assignment is respected, and updates the startup log to reflect the actual bound port.
• local-emulator/Dockerfile: Drops the server.replace('server.listen(8080)', ...) string-patch that stopped matching after the listen callback was added; the two unrelated replaces (fs/promises import, node_modules symlink) are preserved.

Confidence Score: 4/5

Safe to merge — directly fixes the port collision that caused the emulator smoke test to fail.

The core fix is correct and targeted: the source now reads process.env.PORT before the listen call, and the dead string-patch is cleanly removed. The only minor gap is that EXPOSE 8080 in the freestyle-mock Dockerfile becomes a documentation mismatch when a non-default port is injected, but this does not affect runtime behaviour.

The EXPOSE directive in docker/dependencies/freestyle-mock/Dockerfile is now slightly misleading since the actual port is environment-driven.

Important Files Changed

Filename	Overview
docker/dependencies/freestyle-mock/Dockerfile	Replaces hardcoded port 8080 with `process.env.PORT
docker/local-emulator/Dockerfile	Removes the now-redundant (and silently no-oping) server.replace('server.listen(8080)', ...) string-patch; the two remaining replaces for fs/promises import and node_modules symlink are unrelated and kept intact.

Sequence Diagram

sequenceDiagram
    participant S as supervisord
    participant FM as freestyle-mock (server.mjs)
    participant QS as qstash

    Note over S: Reads PORT=8180 from supervisord.conf
    S->>FM: "spawn with PORT=8180"
    FM->>FM: "const PORT = process.env.PORT || 8080 → 8180"
    FM->>FM: server.listen(8180)
    Note over FM: Binds :8180 ✓

    S->>QS: spawn (no PORT override)
    QS->>QS: server.listen(8080)
    Note over QS: Binds :8080 ✓ (no collision)

Loading

Comments Outside Diff (1)

1. docker/dependencies/freestyle-mock/Dockerfile, line 383-390 (link)

   The EXPOSE directive still advertises port 8080, but when PORT=8180 (or any non-default value) is injected by supervisord the container actually listens on a different port. This is a documentation mismatch that can confuse docker inspect, port-mapping documentation, and any tooling that reads EXPOSE to discover the service port.

   Prompt To Fix With AI

   This is a comment left during a code review.
   Path: docker/dependencies/freestyle-mock/Dockerfile
   Line: 383-390
   
   Comment:
   The `EXPOSE` directive still advertises port 8080, but when `PORT=8180` (or any non-default value) is injected by supervisord the container actually listens on a different port. This is a documentation mismatch that can confuse `docker inspect`, port-mapping documentation, and any tooling that reads `EXPOSE` to discover the service port.
   
   
   
   How can I resolve this? If you propose a fix, please make it concise.

Prompt To Fix All With AI

Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
docker/dependencies/freestyle-mock/Dockerfile:383-390
The `EXPOSE` directive still advertises port 8080, but when `PORT=8180` (or any non-default value) is injected by supervisord the container actually listens on a different port. This is a documentation mismatch that can confuse `docker inspect`, port-mapping documentation, and any tooling that reads `EXPOSE` to discover the service port.

```suggestion
const PORT = process.env.PORT || 8080;
server.listen(PORT, () => {
  console.log(`freestyle-mock listening on :${PORT} (worker pool size ${POOL_SIZE})`);
});
EOF

# ---- network ----------------------------------------------------------------
EXPOSE ${PORT:-8080}
```

_{Reviews (1): Last reviewed commit: "[Fix] freestyle-mock honors $PORT, drop ..." | Re-trigger Greptile}