Skip to content

Fix app-server agent storage paths#269

Open
Bortlesboat wants to merge 1 commit into
CortexLM:mainfrom
Bortlesboat:fix/app-server-agent-storage
Open

Fix app-server agent storage paths#269
Bortlesboat wants to merge 1 commit into
CortexLM:mainfrom
Bortlesboat:fix/app-server-agent-storage

Conversation

@Bortlesboat
Copy link
Copy Markdown

Summary

  • Store app-server project agents under .cortex/agents and user agents under the OS config-dir cortex/agents path.
  • Use one set of path helpers across list/get/create/update/delete/import.
  • Add regression coverage for project agent create/list discovery via the standard directory.

Tests

  • cargo test -p cortex-app-server project_agent -- --nocapture
  • cargo test -p cortex-app-server api::agents -- --nocapture
  • cargo test -p cortex-app-server --lib -- --skip tools::shell::tests::test_execute_shell
  • cargo check -p cortex-app-server
  • cargo fmt --check --package cortex-app-server
  • git diff --check

Note: full cargo test -p cortex-app-server currently has an unrelated Windows failure in tools::shell::tests::test_execute_shell because echo is a shell built-in rather than a standalone executable.

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

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.

1 participant