Conversation
🧪 BenchmarkShould we run the MCP Gateway benchmark for this PR? React with 👍 to run the benchmark.
Benchmark will run on the next push after you react. |
Release OptionsShould a new version be published when this PR is merged? React with an emoji to vote on the release type:
Current version: Deployment
|
There was a problem hiding this comment.
1 issue found across 12 files
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="apps/mesh/migrations/seeds/demo/connections.ts">
<violation number="1" location="apps/mesh/migrations/seeds/demo/connections.ts:42">
P2: Inconsistent capitalization: `title` should be `"OpenRouter"` to match the capitalization style of other entries in this file.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| }, | ||
| }, | ||
| openrouter: { | ||
| title: "openrouter", |
There was a problem hiding this comment.
P2: Inconsistent capitalization: title should be "OpenRouter" to match the capitalization style of other entries in this file.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At apps/mesh/migrations/seeds/demo/connections.ts, line 42:
<comment>Inconsistent capitalization: `title` should be `"OpenRouter"` to match the capitalization style of other entries in this file.</comment>
<file context>
@@ -0,0 +1,113 @@
+ },
+ },
+ openrouter: {
+ title: "openrouter",
+ description: "OpenRouter App Connection for LLM uses.",
+ icon: "https://assets.decocache.com/decocms/b2e2f64f-6025-45f7-9e8c-3b3ebdd073d8/openrouter_logojpg.jpg",
</file context>
| title: "openrouter", | |
| title: "OpenRouter", |
✅ Addressed in 78aa820
There was a problem hiding this comment.
1 issue found across 1 file (changes from recent commits).
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="apps/mesh/migrations/seeds/demo/orgs/deco-bank.ts">
<violation number="1" location="apps/mesh/migrations/seeds/demo/orgs/deco-bank.ts:738">
P1: Off-hours calculation produces invalid hour values (24-26). Hours must be in range 0-23. Use modulo arithmetic to properly wrap around midnight.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| } else { | ||
| // Off hours: 10pm to 7am (9 hours) | ||
| const offHour = Math.random() * 9; | ||
| hourOffset = offHour < 5 ? 22 + offHour : offHour - 5; // 22-24 or 0-7 |
There was a problem hiding this comment.
P1: Off-hours calculation produces invalid hour values (24-26). Hours must be in range 0-23. Use modulo arithmetic to properly wrap around midnight.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At apps/mesh/migrations/seeds/demo/orgs/deco-bank.ts, line 738:
<comment>Off-hours calculation produces invalid hour values (24-26). Hours must be in range 0-23. Use modulo arithmetic to properly wrap around midnight.</comment>
<file context>
@@ -721,25 +722,33 @@ function generateSyntheticLogs(targetCount: number): MonitoringLog[] {
- randomOffset = 7 * TIME.DAY + Math.random() * 23 * TIME.DAY;
+ // Off hours: 10pm to 7am (9 hours)
+ const offHour = Math.random() * 9;
+ hourOffset = offHour < 5 ? 22 + offHour : offHour - 5; // 22-24 or 0-7
}
</file context>
| hourOffset = offHour < 5 ? 22 + offHour : offHour - 5; // 22-24 or 0-7 | |
| hourOffset = (22 + offHour) % 24; |
What is this contribution about?
Screenshots/Demonstration
Review Checklist
Summary by cubic
Adds a modular demo seed that creates two realistic demo orgs with users, gateways, connections, and monitoring logs for quick demos and testing. Supports SEED=demo to auto-run during migrations with cleanup and a quick start.
New Features
Migration
Written for commit a38de64. Summary will update on new commits.