feat: configure context paths#86
Conversation
|
@garrettladley thanks for the PR, is it OK if I merge this one in first #69 and than you can rebase and add support for directories? |
|
sounds like a plan @kujtimiihoxha |
|
@garrettladley merged it in, also I am wondering if we should add some message above the file context to tell the LLM that the following is instructions/context provided by the user and it should follow rules and instructions or something like that, that might fix #90. Basically something to tell the LLM that it is important to follow these rules and instructions |
|
@kujtimiihoxha updated and cleaned up the processing code |
garrettladley
changed the title
|
updated 😀 |
|
@garrettladley for some reason I am seeing |
Added nil check in GetPersistentShell before accessing shellInstance.isAlive to prevent panic when newPersistentShell returns nil due to shell startup errors. This resolves the "invalid memory address or nil pointer dereference" error that was occurring in the shell tool.
…formance - Introduced `globWithRipgrep` function to perform file globbing using the `rg` (ripgrep) command. - Updated `globFiles` to prioritize ripgrep-based globbing and fall back to doublestar-based globbing if ripgrep fails. - Added logic to handle ripgrep command execution, output parsing, and filtering of hidden files. - Ensured results are sorted by path length and limited to the specified maximum number of matches. - Modified imports to include `os/exec` and `bytes` for ripgrep integration.
garrettladley
force-pushed
the
feat/configure-context-files
branch
from
3193c48 to
14f627c
Compare
|
@kujtimiihoxha how about now? |
|
@garrettladley Thanks for the PR. Something weird is going on with the commits but I just squash and merge 🤷 |
|
With this commit opencode exits without showing the TUI. Only when there is a ./OpenCode.md present. |
|
uff ok checking |
|
@possibilities fix pushed |
|
Thanks, confirmed |
|
@possibilities I also just pushed a change that prevents duplicate context entries on macos OpenCode.md OPENCODE.md and any variant will be treated the same because of case insensitive paths, the change now prevents it to be added multiple times |
5 participants
implements the feature requested by #59