Skip to content

fix: stage absolute project path in TUI silent notification - #1634

Merged
avoidwork merged 1 commit into
mainfrom
fix/tui
Nov 1, 2026
Merged

avoidwork merged 1 commit into
mainfrom
fix/tui

Conversation

@avoidwork

Copy link
Copy Markdown
Owner

Description

handleSelectProject in src/tui/app.js now stages the silent notification message using the absolute project path (We are working in ${projectPath}.) instead of the relative projects/<name> form. This ensures the agent's downstream path resolution (e.g. scanAgents) is not relative to a mutated process.cwd().

Type of Change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactor (no functional changes)
  • Performance improvement
  • CI / build / tooling

Testing

The pre-commit hook ran npm run fix (oxlint/oxfmt) and npm run coverage (full node --test suite); both passed and the commit landed cleanly. No new tests were authored for this one-line change — N/A.

Coverage

  • Line coverage maintained

Checklist

  • npm run lint passes
  • Tests pass with maintained line coverage
  • No forbidden patterns used
  • Conventional Commit style applied

handleSelectProject now stages the silent notification using the absolute
projectPath ("We are working in ${projectPath}.") instead of the relative
"projects/<name>" form, so downstream path resolution (e.g. scanAgents) is
not relative to a mutated process.cwd().
@avoidwork avoidwork added the bug Something isn't working label Nov 1, 2026
@avoidwork avoidwork self-assigned this Nov 1, 2026
@avoidwork
avoidwork merged commit bb8a0bf into main Nov 1, 2026
3 checks passed
@avoidwork
avoidwork deleted the fix/tui branch November 1, 2026 20:05
@avoidwork avoidwork mentioned this pull request Nov 1, 2026
6 of 12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant