Skip to content

Recognize CircleCI dependency auditing in App Doctor - #8559

Closed
jplhomer wants to merge 1 commit into
joshlarson/app-doctor-dependency-auditing-gitlabfrom
joshlarson/app-doctor-dependency-auditing-circleci
Closed

jplhomer wants to merge 1 commit into
joshlarson/app-doctor-dependency-auditing-gitlabfrom
joshlarson/app-doctor-dependency-auditing-circleci

Conversation

@jplhomer

@jplhomer jplhomer commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

The dependency-auditing check introduced in #8556 should also recognize auditing configured in CircleCI.

WHAT is this pull request doing?

Extends the existing check to inspect .circleci/config.yml and .circleci/config.yaml for scanner commands and Snyk orb scans in workflow-invoked jobs. Reuses the shared discovery safety checks and bounded command analysis without executing CI commands or making network requests.

Uses CircleCI-specific Zod schemas for invoked jobs, steps, and selected scanner inputs. Includes recognition, discovery, and validation-boundary tests. Unused jobs and orbs do not count as evidence; unsupported invoked commands and checkout paths remain unresolved unless separate recognized evidence covers the app.

How to manually test your changes?

pnpm shopify app doctor --path /path/to/circleci-app --skip-instructions
pnpm shopify app doctor --path /path/to/circleci-app --json --blocking low

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • The change is user-facing — I've identified the correct bump type and added a changeset

@jplhomer
jplhomer added this pull request to stack #8557 September 15, 2026 18:24
@github-actions github-actions Bot added the Area: @shopify/app @shopify/app package issues label Sep 15, 2026
Co-authored-by: AI (Pi/GPT-6 Astra) <noreply@pi.dev>
@jplhomer
jplhomer force-pushed the joshlarson/app-doctor-dependency-auditing-circleci branch from 8b20169 to 023e861 Compare September 15, 2026 19:02
@jplhomer

Copy link
Copy Markdown
Contributor Author

Superseded by the simplified file-presence check in #8556. We no longer inspect CircleCI workflows or shell commands; standard Dependabot/Renovate configuration filenames are sufficient. The branch is retained for reference.

@jplhomer jplhomer closed this Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: @shopify/app @shopify/app package issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant