feat(coder-labs/modules/copilot): agentapi state persistence and maintenance#799
feat(coder-labs/modules/copilot): agentapi state persistence and maintenance#79935C4n0r wants to merge 14 commits into
Conversation
|
|
||
| set -euo pipefail | ||
|
|
||
| export PATH="$HOME/.local/bin:$PATH" |
There was a problem hiding this comment.
Should we not check if it exists before prepending the $PATH?
| } | ||
|
|
||
| setup_copilot_config() { | ||
| export XDG_CONFIG_HOME="${XDG_CONFIG_HOME:-$HOME/.config}" |
There was a problem hiding this comment.
@DevelopmentCats I changed the paths here, any reason why we'd want to keep it as $HOME/.config ?
There was a problem hiding this comment.
Yeah, we have had a lot of modules that store their configuration, and logs in random places like the home root, or /tmp.
There has been a push to normalize this across all modules. so you are probably fine to just leave as is and I will take care of it in another PR
|
open for review, but I think we can address this in the PR too: #741 |
|
agentapi module is no longer maintained due to tasks being deprecated. Tasks Migration Guide. All the tasks and agentapi related coupling will be removed once the module is migrated. |
3 participants
Description
Type of Change
Module Information
Path:
registry/coder-labs/modules/copilotNew version:
v0.4.0Breaking change: [ ] Yes [x] No
Testing & Validation
bun test)bun fmt)Merge after: coder/agentapi#203