Open
Conversation
There was a problem hiding this comment.
3 issues found across 12 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="skills/inngest-functions.md">
<violation number="1" location="skills/inngest-functions.md:41">
P3: The trigger examples use `;` inside object literals, which is invalid syntax. Use commas so the documentation shows valid code that can be copied into functions.</violation>
</file>
<file name="skills/sidecar-management.md">
<violation number="1" location="skills/sidecar-management.md:21">
P2: The doc tells users to import `client.js`, but this repo requires `.ts` extensions for ESM and the file is `client.ts`. This guidance will cause sidecar functions to fail to load with a module-not-found error.</violation>
</file>
<file name="src/lib/skills.ts">
<violation number="1" location="src/lib/skills.ts:16">
P2: Frontmatter parsing only supports LF newlines; CRLF markdown files will be skipped from the skills index.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| filePath: string; | ||
| } | ||
|
|
||
| const FRONTMATTER_RE = /^---\n([\s\S]*?)\n---/; |
There was a problem hiding this comment.
P2: Frontmatter parsing only supports LF newlines; CRLF markdown files will be skipped from the skills index.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At src/lib/skills.ts, line 16:
<comment>Frontmatter parsing only supports LF newlines; CRLF markdown files will be skipped from the skills index.</comment>
<file context>
@@ -0,0 +1,61 @@
+ filePath: string;
+}
+
+const FRONTMATTER_RE = /^---\n([\s\S]*?)\n---/;
+
+function parseFrontmatter(content: string): Record<string, string> {
</file context>
|
|
||
| ```typescript | ||
| { | ||
| triggers: [{ cron: "0 */6 * * *" }]; |
There was a problem hiding this comment.
P3: The trigger examples use ; inside object literals, which is invalid syntax. Use commas so the documentation shows valid code that can be copied into functions.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At skills/inngest-functions.md, line 41:
<comment>The trigger examples use `;` inside object literals, which is invalid syntax. Use commas so the documentation shows valid code that can be copied into functions.</comment>
<file context>
@@ -0,0 +1,240 @@
+
+```typescript
+{
+ triggers: [{ cron: "0 */6 * * *" }];
+} // Every 6 hours
+{
</file context>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Optionally run a sidecar Inngest application, where the core agent can write new Inngest functions and have them scheduled by Inngest, so Utah can expand into more automations, async sub-agents and data ingestion workflows.
Summary by cubic
Adds an optional sidecar process that runs an
inngestv4 worker, hot‑loads functions from{workspace.root}/functions, and injects a skills index into the system prompt. This enables scheduling, event-driven automations, and async workflows without touching the core agent.New Features
src/sidecar/sidecar.ts) connects viainngest/connect, respectsAGENT_WORKSPACE, ensures{workspace.root}/functionsexists, loads*.tsdefault exports (skips_*.tsandclient.ts), watches with 2s debounce, hot‑reconnects, and logs JSON with extended traces; built‑in heartbeat every 30m.src/sidecar/client.ts, idutah-sidecar); example cron atworkspace/functions/example-cron.tsusing../../src/sidecar/client.js(.jsrequired).src/lib/skills.ts,src/lib/context.ts); configurableSKILLS_DIRinsrc/config.ts; new skills guides atskills/inngest-functions.mdandskills/sidecar-management.md.utah-agent(src/client.ts); scriptssidecarandsidecar:devadded;.gitignoreupdated for*.plist;inngestbumped to4.1.1.docs/sidecar.md; macOS daemon guide updated with worker/sidecar LaunchAgents and commands; removed obsoletedocs/SUB-AGENTS.md.Migration
.env, then runpnpm sidecar:dev(dev) orpnpm sidecar(prod).{workspace.root}/functions/that default‑exportinngest.createFunction()and import the client from../../src/sidecar/client.js. Avoid top‑level side effects; prefix helper files with_.docs/macos-daemon.md.Written for commit 5702426. Summary will update on new commits.