Integrate PR #28 security and documentation enhancements with yaml-lint reconciliation #30
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR #28 introduced YAML linting, secret detection, and workflow documentation but conflicted with PR #29 (already merged to main) on tooling choice: Python's
yamllintvs Node'syaml-lint. This PR integrates all PR #28 enhancements while maintaining main'syaml-lintapproach.Security Enhancements
.env.example: Added security warnings with crypto-based secret generation commands.husky/pre-commit: Added regex-based secret detection (API keys, tokens, passwords) and.envfile blockingSECURITY.md: Added environment file management section covering rotation, log sanitization, and secret management servicesDocumentation
Added three comprehensive guides:
docs/workflow-maintenance-guide.md: npm script reference, YAML validation procedures, Node version strategy, troubleshootingdocs/copilot-codex-configuration-guide.md: AI pair-programming standards, workflow examples, security defaultsdocs/IMPLEMENTATION_SUMMARY.md: Change summary with reconciliation notesUpdated existing docs:
README.md: Complete npm scripts list (lint:yaml,test:run,check,audit)yamllinttoyaml-lintCI/CD Fixes
.github/workflows/ci.yml:npm audit --audit-level=moderate→npm run audit(uses audit-ci config)[ main, develop ]→[main, develop].github/workflows/copilot-task.yml: Removed trailing spacesAll YAML files validated with
npm run lint:yaml.Known Pre-existing Issue
CI workflow uses Jest commands but project uses Vitest (documented in IMPLEMENTATION_SUMMARY.md for future fix).
Original prompt
This pull request was created from Copilot chat.
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.