Skip to content

chore: centralize graphify and copilot settings - #56

Open
diegoitaliait wants to merge 6 commits into
mainfrom
copilot-config-graphify-settings
Open

chore: centralize graphify and copilot settings#56
diegoitaliait wants to merge 6 commits into
mainfrom
copilot-config-graphify-settings

Conversation

@diegoitaliait

Copy link
Copy Markdown
Contributor

Summary

Move repository-local Graphify guidance to the root AGENTS.md policy owner and configure VS Code Copilot instruction loading explicitly.

Scope

  • Terraform modules
  • IDVH product configuration (IDVH/00_product_configs/*)
  • Repository automation (.github/*, including workflows/prompts/skills/instructions)
  • Scripts (.scripts/*, IDVH/.scripts/*, Bash/Python helpers)
  • Documentation and governance (*.md, CODEOWNERS, .gitignore)
  • Other

Changes

  • Added the ## graphify section to the repository-root AGENTS.md.
  • Added .vscode/settings.json with chat.useInstructionFiles: false and chat.useAgentsMdFile: true.
  • Left .github/copilot-instructions.md unchanged because the branch base already has no Graphify section.

Validation

  • JSON parsing and exact settings assertions: passed.
  • Graphify present in root AGENTS.md and absent from .github/copilot-instructions.md: passed.
  • .github/scripts/validate-copilot-customizations.sh --scope root --mode strict: passed with one warning because .github/workflows is absent.
  • git diff HEAD^ HEAD --check: passed.

Security and Compliance

  • Least privilege preserved (IAM roles/policies/resources reviewed)
  • No hardcoded secrets/tokens
  • External dependencies pinned appropriately (providers/modules/actions)
  • Actions pinned to full-length SHA (if workflows changed)
  • Data/network impact reviewed
  • Breaking changes reviewed and communicated

Risk and Rollback

  • Risk level: Low
  • Blast radius: Repository-local Copilot loading behavior and agent guidance only; no Terraform modules, cloud resources, or runtime code are changed.
  • Rollback plan: Revert commit 61d3677a0f813fc800f64d04a2c014647e59f20b or remove the branch changes.

Related Links

  • Issue: N/A
  • Docs: N/A
  • Runbook: N/A

Reviewer Notes

Review AGENTS.md and .vscode/settings.json first. The pre-existing unrelated work on the local update-review-resources branch was preserved outside this PR.

- Updated pre-commit hooks to the latest versions for improved validation and security checks.
- Added new hooks for YAML and JSON validation, Python linting with Ruff, and shell script linting with ShellCheck.
- Removed deprecated hooks and organized the configuration for better clarity.
- Updated Python version from 3.13 to 3.13.9 for consistency across environments.

docs: add AGENTS.local.md for repository-local policy

- Created AGENTS.local.md to define repository-specific policies and guidelines for maintaining standards and governance.

docs: revise AGENTS.md to clarify repository operating core

- Updated AGENTS.md to streamline instructions for coding agents, emphasizing the importance of user alignment and validation paths.
- Added sections on operating principles, authoring defaults, and tactical defaults to guide agent behavior.

refactor: remove IDVH_CODE_REVIEW.md

- Deleted IDVH_CODE_REVIEW.md as it was no longer relevant to the current repository structure and practices.
…8.0; refactor variable validation and formatting for consistency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant