Skip to content

feat(cli): run the ingest gateway's AI stamping on local traces - #1178

Open
JeremyFunk wants to merge 2 commits into
mainfrom
feat/cli-ai-session-stamps
Open

feat(cli): run the ingest gateway's AI stamping on local traces#1178
JeremyFunk wants to merge 2 commits into
mainfrom
feat/cli-ai-session-stamps

fix(cli): keep a batch the AI stamping traps on, and bound its memory

76179a4
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review completed Sep 30, 2026 in 6m 19s

Confidence 3/5 · No issues found

🟡 Confidence 3/5 · needs attention
quality 100/100 · no findings · tests partial · risk low

Warning

This review ended early; what follows is what it established.

Repoints the AI-stamping doc comments from apps/ingest/src/ai_session to the new apps/ingest/crates/ai-session layout and wires that crate's wasm32 build into the CLI test lane, local binary build and cargo cache key. Build plumbing only; no runtime behavior changes.

  • Comment paths in packages/domain/packages/query-engine-integrations now cite apps/ingest/crates/ai-session/src/*.rs
  • turbo.json adds @maple/cli#test inputs covering apps/ingest/crates/** and Cargo.*
  • mise.toml rust tool installs the wasm32-unknown-unknown target
  • scripts/build-local-binary.sh runs build:ai-stamp before bun build --compile
What was checked
  • Referenced files exist: apps/ingest/crates/ai-session/src/lib.rs, usage.rs, facts.rs
  • The CLI test lane gets rust: install_args keyed on contains(matrix.filters, '@maple/cli') (ci.yml:371)
  • apps/ingest keeps its own exclusion from the TS test matrix (plan-tests.py:53), so no Rust suite is duplicated

76179a4 · Updated on every push. Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.

View more details on Maple Review Bot