actions list/info: add trust level, publisher, paths, policies, and compatibility metadata#435
actions list/info: add trust level, publisher, paths, policies, and compatibility metadata#435Copilot wants to merge 3 commits into
Conversation
|
Meticulous was unable to execute a test run for this PR because the most recent commit is associated with multiple PRs. To execute a test run, please try pushing up a new commit that is only associated with this PR. Last updated for commit |
vu1nz Security Review0 finding(s) in PR #435 No security issues found. Full AI AnalysisAfter thoroughly reviewing this pull request, I have analyzed the code changes for potential security vulnerabilities including SQLi, XSS, RCE, command injection, hardcoded secrets, IDOR, auth/authz flaws, CSRF, SSRF, insecure crypto, path traversal, dependency risks, and CI/CD supply-chain issues. Security Analysis ResultsNO security issues were found in this pull request. Analysis SummaryThis PR appears to be adding functionality to display and evaluate "trust levels" for action packs in a CLI tool. The changes include:
Key Security Considerations Reviewed
The changes enhance the security posture of the tool by providing visibility into action pack security configurations, which is a positive security enhancement rather than introducing vulnerabilities. |
actions listwas missingpublisherandactions info/showwas missing several metadata fields called out in the PRD: trust level, pack path, compatibility, policies, and full security details.Changes
actions listpublisheradded to every row (text +--json)searchnow also matches on publisher textactions info/actions showtrust level— derived field (high/medium/low) from the manifest'ssecurityblock:high:leastPrivilegePermissions=true+pinThirdPartyActions=required+allowPullRequestTarget=falsemedium:leastPrivilegePermissions=true+allowPullRequestTarget=falselow: anything elsepack path—CatalogEntry.packDir(absolute path to the pack on disk)--jsonoutput now includespackDirandtrustLevelNew export
Tests — 5 new unit tests covering
deriveTrustLevellogic and subcommand presence.Example output (
actions info node-pnpm-ci)