Skip to content

chore(ops): harden cAPI networking to use canonical golden bible ports - #21

Merged
reprewindai-dev merged 1 commit into
mainfrom
ops-capi-hardening
Jul 25, 2026
Merged

chore(ops): harden cAPI networking to use canonical golden bible ports#21
reprewindai-dev merged 1 commit into
mainfrom
ops-capi-hardening

Conversation

@reprewindai-dev

@reprewindai-devreprewindai-dev commented Jul 25, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Bug Fixes
    • Updated backend health-check and signature-minting connections to use the correct service ports.
    • Improved generated MCP proxy links to use the production application address when no URL is configured.

@vercel

vercelBot commented Jul 25, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
c-apiReadyReadyPreview, CommentJul 25, 2026 5:15pm
veklom-id-59uwReadyReadyPreview, CommentJul 25, 2026 5:15pm

@coderabbitai

coderabbitaiBot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2fd65f10-5942-4a4c-b710-20b5e50bfae1

📥 Commits

Reviewing files that changed from the base of the PR and between af93aa9 and 850a91a.

📒 Files selected for processing (2)
  • src/app/api/ops/validate/route.ts
  • src/lib/covenant/dynamic-mcp.ts

📝 Walkthrough

Walkthrough

The validation route now targets updated backend ports for health checks and Gnomledger minting. The MCP proxy fallback URL now uses control.veklom.com instead of localhost:3002.

Changes

Runtime endpoint updates

Layer / File(s)Summary
Endpoint routing updates
src/app/api/ops/validate/route.ts, src/lib/covenant/dynamic-mcp.ts
The health-check target changes to port 8002, the Gnomledger target changes to port 8001, and the MCP proxy fallback changes to https://control.veklom.com.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check nameStatusExplanationResolution
Docstring Coverage⚠️ WarningDocstring coverage is 50.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 nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title matches the main change: hardening ops networking and port/host configuration.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ 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 ops-capi-hardening

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

src/app/api/ops/validate/route.ts

Oops! Something went wrong! :(

ESLint: 10.7.0

TypeError [ERR_IMPORT_ATTRIBUTE_MISSING]: Module "file:///.eslintrc.json?mtime=1784999759822" needs an import attribute of "type: json"
at validateAttributes (node:internal/modules/esm/assert:88:15)
at defaultLoadSync (node:internal/modules/esm/load:164:3)
at #loadAndMaybeBlockOnLoaderThread (node:internal/modules/esm/loader:776:12)
at #loadSync (node:internal/modules/esm/loader:796:49)
at ModuleLoader.load (node:internal/modules/esm/loader:762:26)
at ModuleLoader.loadAndTranslate (node:internal/modules/esm/loader:504:31)
at #getOrCreateModuleJobAfterResolve (node:internal/modules/esm/loader:555:36)
at afterResolve (node:internal/modules/esm/loader:603:52)
at ModuleLoader.getOrCreateModuleJob (node:internal/modules/esm/loader:609:12)
at node:internal/modules/esm/loader:628:32

src/lib/covenant/dynamic-mcp.ts

Oops! Something went wrong! :(

ESLint: 10.7.0

TypeError [ERR_IMPORT_ATTRIBUTE_MISSING]: Module "file:///.eslintrc.json?mtime=1784999759822" needs an import attribute of "type: json"
at validateAttributes (node:internal/modules/esm/assert:88:15)
at defaultLoadSync (node:internal/modules/esm/load:164:3)
at #loadAndMaybeBlockOnLoaderThread (node:internal/modules/esm/loader:776:12)
at #loadSync (node:internal/modules/esm/loader:796:49)
at ModuleLoader.load (node:internal/modules/esm/loader:762:26)
at ModuleLoader.loadAndTranslate (node:internal/modules/esm/loader:504:31)
at #getOrCreateModuleJobAfterResolve (node:internal/modules/esm/loader:555:36)
at afterResolve (node:internal/modules/esm/loader:603:52)
at ModuleLoader.getOrCreateModuleJob (node:internal/modules/esm/loader:609:12)
at node:internal/modules/esm/loader:628:32


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.

@reprewindai-dev
reprewindai-dev merged commit fed3017 into mainJul 25, 2026
3 of 5 checks passed
Sign up for freeto 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

@reprewindai-dev