Skip to content

fix: make vercel api imports runtime-safe and add landing page#14

Merged
dan13ram merged 2 commits intomasterfrom
refactor/vercel-deploy
Apr 2, 2026
Merged

fix: make vercel api imports runtime-safe and add landing page#14
dan13ram merged 2 commits intomasterfrom
refactor/vercel-deploy

Conversation

@dan13ram
Copy link
Copy Markdown
Collaborator

@dan13ram dan13ram commented Apr 2, 2026

Summary by CodeRabbit

  • New Features

    • Added a landing page displaying service status and info about scheduled notifications.
  • Chores

    • Consolidated import paths project-wide and added explicit TypeScript dev dependency.
  • Minor fixes

    • Small runtime-related adjustments: timestamp formatting, retry parsing, cache initialization, and chain/attestation handling to improve stability and consistency.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 2, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
builder-bot Ignored Ignored Preview Apr 2, 2026 11:10am

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 2, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0b7b2216-124b-4e3b-907a-b655ef216b4c

📥 Commits

Reviewing files that changed from the base of the PR and between 1103b64 and ba051d6.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (6)
  • package.json
  • src/commands/process/proposals.ts
  • src/commands/queues/consume.ts
  • src/queue.ts
  • src/services/eas/get-propdate-attestations.ts
  • src/services/eas/index.ts

📝 Walkthrough

Walkthrough

Migrates many alias-based imports (@/…) to relative ../src/... or ../../src/... paths across API cron endpoints and health check; adds a new index.html landing page; pins/upgrades TypeScript-related dev dependencies; and makes small local type/coercion tweaks in several src modules. No exported APIs or handler signatures changed.

Changes

Cohort / File(s) Summary
Cron API Import Migration
api/cron/cleanup.ts, api/cron/consume-queue.ts, api/cron/process-invites.ts, api/cron/process-propdates.ts, api/cron/process-proposals.ts
Replaced @/… alias imports with relative ../../src/.../../../src/services/... paths; added node:crypto randomUUID import in proposal/propdate processors; reorganized some multiline named imports (e.g., job lock utilities).
API Health Import Migration
api/health.ts
Switched imports for env, prisma, and isAuthorizedCronRequest from @/ alias to relative ../src/... paths.
Static Landing Page
index.html
New HTML landing page added with styled content referencing the /api/health endpoint.
Tooling / DevDependencies
package.json
Added typescript@5.9.3 to devDependencies and updated TypeScript ESLint packages (parser/eslint versions updated).
Runtime/coercion small tweaks
src/commands/process/proposals.ts, src/commands/queues/consume.ts, src/queue.ts, src/services/eas/get-propdate-attestations.ts, src/services/eas/index.ts
Minor internal changes removing explicit Number(...) coercions in several places (cache init, timestamp handling, maxRetries parsing, chainId/messageType handling). No API/signature changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 I hopped through imports, line by line,
From @/ alleys to relative vine.
A page now greets at /api/health gate,
TypeScript tuned, the build feels great.
Little changes, tidy and spry — we celebrate! 🎉

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: refactoring imports to use relative paths for runtime safety and adding a new landing page HTML file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/vercel-deploy

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 and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@package.json`:
- Around line 80-81: package.json currently pins "typescript" at 5.9.3 while
"typescript-eslint" is 8.15.0 which is incompatible; update package.json so the
linter packages and parser match the TypeScript version: either downgrade
"typescript" to a <5.7.0 version, or (preferred) upgrade "typescript-eslint" and
"@typescript-eslint/parser" to at least 8.26.0 (or a newer compatible release)
so the parser supports TypeScript 5.9; modify the "typescript-eslint" entry and
add/update "@typescript-eslint/parser" to the same compatible semver and run
your lockfile update (npm/yarn pnpm install) to apply the changes.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 171c93c4-97ec-4ad6-9fa8-55551b596f22

📥 Commits

Reviewing files that changed from the base of the PR and between 0ce8358 and 1103b64.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (8)
  • api/cron/cleanup.ts
  • api/cron/consume-queue.ts
  • api/cron/process-invites.ts
  • api/cron/process-propdates.ts
  • api/cron/process-proposals.ts
  • api/health.ts
  • index.html
  • package.json

@dan13ram dan13ram merged commit c609b4c into master Apr 2, 2026
3 of 4 checks passed
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