hermes - 💡(How to fix) Fix time_reconciler: add Timing adapter and approval-only executor

Official PRs (…)
ON THIS PAGE

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
RAW_BUFFERClick to expand / collapse

Parent: #29631

Goal

Implement a Timing provider adapter and approval-only executor for time_reconciler.

Scope

  • Add Timing adapter methods for:
    • list projects/project hierarchy
    • fetch activity blocks
    • fetch existing time entries
    • create approved time entries
  • Credentials must come from configurable env var names, e.g. TIMING_API_KEY, and never be serialized into artifacts.
  • Executor must consume only an explicit approved manifest, not raw LLM proposals.
  • Executor must write an audit result with created/updated entry references.

Acceptance Criteria

  • Adapter can be unit-tested with mocked HTTP responses.
  • Executor refuses unapproved proposal manifests.
  • No secrets appear in logs/artifacts.
  • Timing-specific code is isolated behind provider interface.
  • Tests use mocked/synthetic Timing data only.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

hermes - 💡(How to fix) Fix time_reconciler: add Timing adapter and approval-only executor