Menu

#340 fix(dev): clean startup, worklog pipeline wiring, and a11y capture gate

closed
nobody
None
2026-06-25
2026-06-25
Anonymous
No

Originally created by: adityaharishch

Summary

  • dev-start.sh: clear ui/.next cache on each run so stale module references (e.g. deleted instrumentation.ts) don't break beforeDevCommand; add ui/node_modules preflight so the error surfaces clearly; kill stale Meridian Dev.app process before relaunching
  • install.sh: add npm install in ui/ during dev-path setup so ui/node_modules is present when npm run tauri dev runs beforeDevCommand
  • src/main.rs: wire worklog_pipeline::run_loop (calls /worklog_hour hourly) instead of the decommissioned pm_worklog::run_loop (was calling /synthesise_worklog → 404)
  • tray/src-tauri/src/commands/health.rs: gate the a11y-helper trust check behind #[cfg(not(feature = "capture"))] — in-process capture builds don't use the a11y-helper binary, so the banner must never fire
  • services/agents/observability.py: add instrument_agno() and current_traceparent() for W3C traceparent propagation from agno-based pipelines

Test plan

  • [ ] Fresh clone → bash install.sh (dev path) → bash dev-start.sh — no next: No such file or directory error
  • [ ] Re-run bash dev-start.shui/.next is cleared, no stale module error
  • [ ] Tauri dev build with --features capture — health check returns no a11y_helper_trusted field, a11y banner absent
  • [ ] MLX server logs — no more POST /synthesise_worklog 404; /worklog_hour called on the hour instead

Related

Tickets: #347

Discussion

  • Anonymous

    Anonymous - 2026-06-25

    Originally posted by: coderabbitai[bot]

    [!IMPORTANT]

    Review skipped

    Auto reviews are disabled on base/target branches other than the default branch.

    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.


    ⚙️ Run configuration

    Configuration used: Organization UI

    Review profile: ASSERTIVE

    Plan: Pro Plus

    Run ID: 6ba45fb2-becd-4ce5-a6af-13a614e67a3b

    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 - [ ] Commit unit tests in branch `fix/dev-env-startup`

    Comment @coderabbitai help to get the list of available commands.

     
  • Anonymous

    Anonymous - 2026-06-25

    Originally posted by: adityaharishch

    Fixed the current_traceparent finding — now uses TraceContextTextMapPropagator().inject(carrier) instead of hand-rolling the W3C format string, matching the extraction pattern and handling tracestate correctly.

     
  • Anonymous

    Anonymous - 2026-06-25

    Ticket changed by: Akarsh-Hegde

    • status: open --> closed
     

Log in to post a comment.