Summary
The copyright header validation script Test-CopyrightHeaders.ps1 maintains an allow-list of file extensions it checks for SPDX license headers. Python .py files are not in this list, meaning Python scripts can merge without the required copyright header that all other source files must carry.
This is a one-line change that unblocks all subsequent Python CI work by ensuring Python files are held to the same SPDX compliance standards as PowerShell and shell scripts.
Context
PR #868 introduces the first Python skill (PowerPoint automation) with multiple .py files. Without this change, those files bypass copyright header validation entirely. The existing npm run validate:copyright command and CI workflow will automatically pick up the new extension once added.
Prior work in #634 (closed) established the CI coverage expansion pattern for skills. This issue extends that pattern to Python source files.
Changes Required
| File |
Change |
scripts/linting/Test-CopyrightHeaders.ps1 |
Add *.py to the file extension array |
Acceptance Criteria
OSSF Impact
Low direct impact, but satisfies the license_per_file_documentation criterion by ensuring Python files carry SPDX headers.
Dependencies
None. This is a standalone change with no prerequisites.
Related
Summary
The copyright header validation script
Test-CopyrightHeaders.ps1maintains an allow-list of file extensions it checks for SPDX license headers. Python.pyfiles are not in this list, meaning Python scripts can merge without the required copyright header that all other source files must carry.This is a one-line change that unblocks all subsequent Python CI work by ensuring Python files are held to the same SPDX compliance standards as PowerShell and shell scripts.
Context
PR #868 introduces the first Python skill (PowerPoint automation) with multiple
.pyfiles. Without this change, those files bypass copyright header validation entirely. The existingnpm run validate:copyrightcommand and CI workflow will automatically pick up the new extension once added.Prior work in #634 (closed) established the CI coverage expansion pattern for skills. This issue extends that pattern to Python source files.
Changes Required
scripts/linting/Test-CopyrightHeaders.ps1*.pyto the file extension arrayAcceptance Criteria
*.pyis included in theTest-CopyrightHeaders.ps1extension listnpm run validate:copyrightscans Python files and reports missing headersOSSF Impact
Low direct impact, but satisfies the
license_per_file_documentationcriterion by ensuring Python files carry SPDX headers.Dependencies
None. This is a standalone change with no prerequisites.
Related