Objective
Fix the 156 ShellCheck SC1003 occurrences across 23 workflow source files by replacing single-quoted domain lists with double-quoted strings in awf --allow-domains arguments.
Context
From static analysis discussion #17844: The daily static analysis scan reports 156 shellcheck:SC1003 findings (info severity) across 23 agentic workflows. ShellCheck warns because single-quoted strings can mask escape sequences. Double-quoting domain lists is the correct fix since wildcards like *.githubusercontent.com are command arguments, not shell globs.
Approach
- Find all
.md source files in .github/workflows/ that use awf --allow-domains '...'
- Replace single-quoted domain lists with double-quoted strings:
'*.domain.com,...' → "*.domain.com,..."
- After fixing all 23 files, run
make recompile to regenerate .lock.yml files
- Verify the SC1003 count drops to 0 with
actionlint .github/workflows/*.lock.yml
Files to Modify (source .md files only)
.github/workflows/copilot-cli-deep-research.md
.github/workflows/daily-compiler-quality.md
.github/workflows/daily-doc-updater.md
.github/workflows/daily-file-diet.md
.github/workflows/daily-mcp-concurrency-analysis.md
.github/workflows/daily-syntax-error-quality.md
.github/workflows/daily-testify-uber-super-expert.md
.github/workflows/delight.md
.github/workflows/developer-docs-consolidator.md
.github/workflows/discussion-task-miner.md
.github/workflows/glossary-maintainer.md
.github/workflows/go-fan.md
.github/workflows/go-logger.md
.github/workflows/instructions-janitor.md
.github/workflows/layout-spec-maintainer.md
.github/workflows/semantic-function-refactor.md
.github/workflows/sergo.md
.github/workflows/step-name-alignment.md
.github/workflows/typist.md
.github/workflows/ubuntu-image-analyzer.md
.github/workflows/unbloat-docs.md
.github/workflows/workflow-skill-extractor.md
.github/workflows/ci-doctor.md
Example Fix
# Before (triggers SC1003):
run: |
sudo -E awf --allow-domains '*.githubusercontent.com,api.github.com,models.github.com' \
--other-flags
# After (correct):
run: |
sudo -E awf --allow-domains "*.githubusercontent.com,api.github.com,models.github.com" \
--other-flags
IMPORTANT: Fix .md source files only. Run make recompile after all fixes to regenerate .lock.yml files. Run make agent-finish before committing.
Acceptance Criteria
Generated by Plan Command for issue #discussion #17844
Objective
Fix the 156 ShellCheck SC1003 occurrences across 23 workflow source files by replacing single-quoted domain lists with double-quoted strings in
awf --allow-domainsarguments.Context
From static analysis discussion #17844: The daily static analysis scan reports 156
shellcheck:SC1003findings (info severity) across 23 agentic workflows. ShellCheck warns because single-quoted strings can mask escape sequences. Double-quoting domain lists is the correct fix since wildcards like*.githubusercontent.comare command arguments, not shell globs.Approach
.mdsource files in.github/workflows/that useawf --allow-domains '...''*.domain.com,...'→"*.domain.com,..."make recompileto regenerate.lock.ymlfilesactionlint .github/workflows/*.lock.ymlFiles to Modify (source
.mdfiles only).github/workflows/copilot-cli-deep-research.md.github/workflows/daily-compiler-quality.md.github/workflows/daily-doc-updater.md.github/workflows/daily-file-diet.md.github/workflows/daily-mcp-concurrency-analysis.md.github/workflows/daily-syntax-error-quality.md.github/workflows/daily-testify-uber-super-expert.md.github/workflows/delight.md.github/workflows/developer-docs-consolidator.md.github/workflows/discussion-task-miner.md.github/workflows/glossary-maintainer.md.github/workflows/go-fan.md.github/workflows/go-logger.md.github/workflows/instructions-janitor.md.github/workflows/layout-spec-maintainer.md.github/workflows/semantic-function-refactor.md.github/workflows/sergo.md.github/workflows/step-name-alignment.md.github/workflows/typist.md.github/workflows/ubuntu-image-analyzer.md.github/workflows/unbloat-docs.md.github/workflows/workflow-skill-extractor.md.github/workflows/ci-doctor.mdExample Fix
IMPORTANT: Fix
.mdsource files only. Runmake recompileafter all fixes to regenerate.lock.ymlfiles. Runmake agent-finishbefore committing.Acceptance Criteria
.mdfiles updated to use double-quoted domain listsmake recompileruns without errorsmake agent-finishpasses (build + test + lint)