fix(runtime): handle IGNORE action fallback and tolerate upstream IGNORE

[paulf280-ui]

Summary

• Fixes IGNORE action handling in the agent runtime — when the LLM selects IGNORE, the runtime now falls back gracefully rather than erroring or hanging
• Adds diagnostics and tolerance for upstream IGNORE action signals in the bootstrap plugin
• Pins tsup, externalizes tokenizers, and fixes embedding input coercion in @elizaos/core
• Fixes UUID and parsing helpers; makes embedding tests deterministic
• Adds CLAUDE.md with build commands and architecture overview for contributors

Test plan

• Run bun run test:core — all vitest tests pass including new deterministic embedding tests
• Verify agent does not crash/hang when LLM responds with IGNORE action
• Verify bun run build:core succeeds cleanly

🤖 Generated with Claude Code

Greptile Summary

This PR adds core TypeScript files (runtime.ts, embedding.ts, parsing.ts, uuid.ts) to @elizaos/core package that were previously missing from source control. The main changes include:

• IGNORE action handling: Implements graceful fallback when LLM returns IGNORE action (lines 1084-1088 in runtime.ts) — treats it as a no-op instead of throwing "Action not found" errors
• Embedding robustness: Adds input type coercion to handle non-string inputs gracefully, returning empty arrays for empty inputs
• Build configuration: Pins tsup to ^8.5.1 and externalizes native modules (@anush008/tokenizers, onnxruntime-node, sharp) to prevent bundling issues
• Test coverage: Adds deterministic embedding tests with mocked dependencies
• Python additions: Adds agent.py, run_traderbot.py, and trader.json for Python runtime testing; extensively refactors runtime.py (5484 lines changed)

Issues found:

• Type mismatch in embedding.ts:234 where input (type unknown) is passed to retrieveCachedEmbedding which expects string — should pass text instead
• Inconsistent logging using console.error instead of elizaLogger in parsing.ts
• Misleading comment in tsup.config.ts about CommonJS when using ESM format

Confidence Score: 3/5

• This PR has one critical type mismatch bug that must be fixed before merge
• Score reflects a logic bug in embedding.ts:234 where wrong variable type is passed to retrieveCachedEmbedding. This could cause runtime type errors. The IGNORE action handling and other changes are well-implemented, but the type bug needs resolution.
• Pay close attention to packages/core/src/embedding.ts line 234 — type mismatch that needs fixing

Important Files Changed

Filename	Overview
packages/core/src/runtime.ts	Added IGNORE action fast-path at lines 1084-1088 to gracefully handle IGNORE actions without throwing errors
packages/core/src/embedding.ts	Added input coercion for robustness, but type mismatch at line 234 where input is passed instead of text to retrieveCachedEmbedding
packages/core/src/parsing.ts	Added comprehensive JSON parsing utilities, but uses console.error instead of elizaLogger at 8 locations (lines 100, 101, 119, 120, 154, 155, 168, 169)
packages/core/tsup.config.ts	Added build config with tokenizers externalized, but misleading comment at line 8 says "targeting CommonJS" when format is "esm"
packages/python/elizaos/runtime.py	Extensive refactoring with 5484 lines changed; difficult to review comprehensively without more context

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    Start[LLM Returns Action] --> Normalize[Normalize Action Name]
    Normalize --> CheckIgnore{Is IGNORE or<br/>contains 'ignore'?}
    CheckIgnore -->|Yes| LogIgnore[Log: Skipping IGNORE action]
    CheckIgnore -->|No| FindAction[Look up action in registry]
    LogIgnore --> Continue[Continue to next message]
    FindAction --> ActionExists{Action found?}
    ActionExists -->|Yes| CheckHandler{Has handler?}
    ActionExists -->|No| LogError[Log: Action not found]
    CheckHandler -->|Yes| Execute[Execute action.handler]
    CheckHandler -->|No| LogNoHandler[Log: No handler]
    LogError --> Continue
    LogNoHandler --> Continue
    Execute --> HandleError{Error occurred?}
    HandleError -->|Yes| LogExecError[Log execution error]
    HandleError -->|No| Success[Action completed]
    LogExecError --> Continue
    Success --> Continue

Loading

_{Last reviewed commit: dbc6b01}

_{(2/5) Greptile learns from your feedback when you react with thumbs up/down!}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

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]

_{13 files reviewed, 3 comments}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

type mismatch — retrieveCachedEmbedding expects string but receives input (type unknown). should pass text instead

Suggested change

	const cachedEmbedding = await retrieveCachedEmbedding(runtime, input);
	const cachedEmbedding = await retrieveCachedEmbedding(runtime, text);

[greptile-apps]

use elizaLogger.error instead of console.error for consistency with the rest of the codebase (also affects lines 119-120, 154-155, 168-169)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

[greptile-apps]

misleading comment — says "targeting CommonJS" but format is esm

Suggested change

	format: ["esm"], // Ensure you're targeting CommonJS
	format: ["esm"], // ESM format for modern Node.js