Skip to content
This repository was archived by the owner on Jul 16, 2026. It is now read-only.

fix(inline-tools): get_core_status reads from workspace state/ not repo root - #105

Open
jgluck96 wants to merge 2 commits into
mainfrom
fix/get-core-status-workspace-path
Open

jgluck96 wants to merge 2 commits into
mainfrom
fix/get-core-status-workspace-path

Conversation

@jgluck96

@jgluck96 jgluck96 commented Jun 4, 2026

Copy link
Copy Markdown

Summary

  • getCoreStatusTool (get_core_status voice tool) constructed the path as join(repoDir, 'core-status.json') — the repo root
  • But core-status.json is written to WORKSPACE_DIR/state/core-status.json per the workspace contract (CLAUDE.md)
  • The file at the repo root never exists, so the tool always returned { status: 'idle', description: 'Core agent is not currently running.' } — even mid-pass when the loop was actively working
  • Fix: join(resolveWorkspace(), 'state', 'core-status.json') — resolveWorkspace is already imported in this file

Test plan

  • Confirmed resolveWorkspace already imported at line 12 of inline-tools.ts
  • Verified the fix reads from the same path the proactive loop writes to ($WORKSPACE/state/core-status.json)
  • tsc type check clean
  • CI

🤖 Generated with Claude Code

jgluck96 and others added 2 commits June 4, 2026 17:04
…po root

getCoreStatusTool was joining repoDir + 'core-status.json', but the
workspace contract writes core-status.json to WORKSPACE_DIR/state/.
The file at repo root never exists, so the tool always returned
{ status: 'idle', description: 'Core agent is not currently running.' }
even when the proactive loop was actively working.

Fix: join(resolveWorkspace(), 'state', 'core-status.json') — same
resolver that every other WORKSPACE_DIR consumer uses. resolveWorkspace
is already imported in this file.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The tool now reads from resolveWorkspace()/state/core-status.json
(not repo-root). Update CORE_STATUS_PATH and ensure state/ dir exists.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

@bassilkhilo-ag2 bassilkhilo-ag2 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean fix. resolveWorkspace() is already imported at line 12; swapping the hardcoded repo-root path to join(resolveWorkspace(), 'state', 'core-status.json') is exactly right per the workspace contract. The test update correctly adds mkdirSync for the state/ dir and switches to the canonical path. Approved.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants