Skip to content

[plan] Fix SC2295: Add proper expansion quoting in ci-doctor workflow #17853

@github-actions

Description

@github-actions

Objective

Fix the SC2295 shellcheck warning in ci-doctor workflow by adding proper quoting to the expansion inside ${..}.

Context

From static analysis discussion #17844: Actionlint reports one shellcheck:SC2295 occurrence in ci-doctor.lock.yml:305. This is a new finding (introduced since 2026-02-22) and the only non-SC1003 actionlint issue remaining. SC2295 warns about expansions inside ${..} that may need quoting to avoid word splitting.

Approach

  1. Open .github/workflows/ci-doctor.md (source file — do NOT edit the .lock.yml)
  2. Locate the shell step around line 305 of the compiled file (search for the expansion pattern)
  3. The fix involves quoting the variable inside the expansion, for example:
    # Before (SC2295):
    echo ${variable/pattern/replacement}
    
    # After (proper quoting):
    echo "${variable/pattern/replacement}"
  4. Run make recompile to regenerate ci-doctor.lock.yml
  5. Verify the fix: actionlint .github/workflows/ci-doctor.lock.yml
  6. Run make agent-finish before committing

Files to Modify

  • .github/workflows/ci-doctor.md (source — find the step that generates line ~305 in the lock file)

Acceptance Criteria

  • SC2295 warning in ci-doctor is resolved
  • make recompile regenerates ci-doctor.lock.yml without errors
  • actionlint .github/workflows/ci-doctor.lock.yml reports 0 findings
  • make agent-finish passes

Generated by Plan Command for issue #discussion #17844

  • expires on Feb 25, 2026, 7:18 AM UTC

Metadata

Metadata

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions