[AVC] Refactor commands and add new audit commands#14886
Open
tjprescott wants to merge 7 commits intomainfrom
Open
[AVC] Refactor commands and add new audit commands#14886tjprescott wants to merge 7 commits intomainfrom
tjprescott wants to merge 7 commits intomainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors the APIView Copilot CLI command surface (renaming/re-grouping several command groups) and adds new audit/reporting commands for examining AI comment feedback and stored memories in the AVC Cosmos DB, while simplifying environment configuration by deriving the App Configuration endpoint from ENVIRONMENT_NAME.
Changes:
- Refactor CLI groups/commands (e.g.,
eval→test,search→kb,metrics→report,app/permissions→ops) and update docs accordingly. - Add new reporting/audit commands (
avc report feedback/avc report memory) and related support logic (including feedback language handling). - Update settings/db/search managers to support environment-aware configuration and instance scoping.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/python-packages/apiview-copilot/src/_settings.py | Resolve App Configuration endpoint from ENVIRONMENT_NAME via a fixed mapping; support per-environment SettingsManager instances. |
| packages/python-packages/apiview-copilot/src/_search_manager.py | Allow SearchManager to be constructed with an explicit environment. |
| packages/python-packages/apiview-copilot/src/_database_manager.py | Key DatabaseManager singletons by environment to support staging/production side-by-side usage. |
| packages/python-packages/apiview-copilot/src/_apiview.py | Extend AI comment feedback query to support language=None (all languages) and include language in results. |
| packages/python-packages/apiview-copilot/scripts/dump_cli_command_tree.py | New helper script to dump the CLI command tree and descriptions. |
| packages/python-packages/apiview-copilot/README.md | Update examples/docs for the refactored CLI groups and env setup. |
| packages/python-packages/apiview-copilot/cli.py | Refactor command groups, add audit/report commands, and add test pytest convenience command. |
| packages/python-packages/apiview-copilot/AGENTS.md | Update operational docs for renamed commands and env setup. |
| packages/python-packages/apiview-copilot/.github/skills/run-evals/SKILL.md | Update skill instructions to use the new avc test eval command path. |
| packages/python-packages/apiview-copilot/.github/skills/dev-setup/SKILL.md | Update dev setup skill instructions to match the new CLI command groups. |
packages/python-packages/apiview-copilot/src/_search_manager.py
Outdated
Show resolved
Hide resolved
023ce03 to
b2327ae
Compare
b2327ae to
8a37856
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR refactors the APIView Copilot CLI command surface (renaming/re-grouping several command groups) and adds new audit/reporting commands for examining AI comment feedback and stored memories in the AVC Cosmos DB, while simplifying environment configuration by deriving the App Configuration endpoint from ENVIRONMENT_NAME.