Skip to content

refactor(chat): remove auto file and dir attachment#200

Merged
cniska merged 5 commits intomainfrom
remove-auto-file-context
Apr 14, 2026
Merged

refactor(chat): remove auto file and dir attachment#200
cniska merged 5 commits intomainfrom
remove-auto-file-context

Conversation

@cniska
Copy link
Copy Markdown
Owner

@cniska cniska commented Apr 14, 2026

Motivation

Auto-inlining @ file and directory contents burned context tokens before the model decided what was relevant. Letting the model pull what it needs through existing tools aligns with the on-demand retrieval pattern already used by memory.

Summary

  • replace file content injection with suggestions via req.suggestions
  • suggest code-scan for parseable code files, file-read for other files, file-find for directories
  • remove isRelevantFileContext and attachment message token budget from agent input
  • remove maxAttachmentMessageTokens from lifecycle policy and constants
  • unresolved @ refs now warn but no longer block the turn
  • export isParseable from code-ops for reuse

Fixes #196

@cniska cniska force-pushed the remove-auto-file-context branch from d2a4cb4 to 48d90ff Compare April 14, 2026 19:26
@cniska cniska force-pushed the remove-auto-file-context branch from 48d90ff to 502612e Compare April 14, 2026 20:04
@cniska cniska changed the title refactor(chat): remove auto file context attachment refactor(chat): remove auto file and dir attachment Apr 14, 2026
@cniska cniska merged commit 39117e3 into main Apr 14, 2026
5 checks passed
@cniska cniska deleted the remove-auto-file-context branch April 14, 2026 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor(chat): remove auto file context attachment

1 participant