Skip to content

chore: synchronize with latest pyproject-template#483

Merged
endavis merged 2 commits intomainfrom
chore/482-sync-pyproject-template
Apr 3, 2026
Merged

chore: synchronize with latest pyproject-template#483
endavis merged 2 commits intomainfrom
chore/482-sync-pyproject-template

Conversation

@endavis
Copy link
Copy Markdown
Owner

@endavis endavis commented Apr 3, 2026

Description

Synchronize project with latest pyproject-template improvements following the AI Sync Checklist.

Related Issue

Addresses #482

Type of Change

  • Code refactoring
  • Chore / maintenance

Changes Made

Phase 3: pyproject.toml

  • Added ruff per-file-ignores for tools/opnsense-parser.py and tools/pyproject_template/*.py

Phase 5: Doit Tasks

  • Added tools/ to lint, format, spell check, and bandit scan targets
  • Cleaned up string concatenation in git.py
  • Removed unnecessary cmd variable in quality.py task_type_check
  • Refactored install.py to use create_install_task() from template framework
  • Refactored install_tools.py: adopted generic framework (get_latest_github_release, install_tool, create_install_task) and reimplemented project-specific tools (age, terraform, opentofu) using shared helpers
  • Auto-formatted infrastructure.py (ruff now covers tools/)

Phase 6: AGENTS.md

  • Added emoji indicators to headings (⚠️ Core Mandate, ⚠️ Mandatory Protocols)
  • Improved Decision Framework table with ✅ ⚠️ 🚫 status icons
  • Added ❌/✅ prefixes to Prohibited/Correct reasoning examples
  • Expanded Task Planning Protocol to numbered checklist
  • Expanded Blocked Protocol with examples
  • Added "What NOT to do" section for Dependabot PRs
  • Added "Temporary Files" section (tmp/agents/<type>/ convention)
  • Typography improvements (→ arrows, ≠ symbol)

Phase 7: Pre-commit

  • Added commented-out exclusion hints for check-added-large-files and detect-private-key

Phase 8: AI Hooks

  • Added commented-out sandbox_mode and network_access hints to .codex/config.toml

Phase 10: Sync

  • Marked sync point at template commit 75cbe9b

Testing

  • doit check passes (all checks)
  • No source code changes — tooling and config only

Checklist

  • My code follows the code style of this project (ran doit format)
  • I have run linting checks (doit lint)
  • I have run type checking (doit type_check)
  • All new and existing tests pass (doit test)
  • My changes generate no new warnings

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark

Details
Benchmark suite Current: 8b0be7d Previous: 482dc64 Ratio
tests/benchmarks/test_placeholder.py::test_import_time 6338.331177465948 iter/sec (stddev: 0.000035300431259638925) 7365.802725990653 iter/sec (stddev: 0.00000874050089396922) 1.16

This comment was automatically generated by workflow using github-action-benchmark.

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.10.

Benchmark suite Current: 8b0be7d Previous: 482dc64 Ratio
tests/benchmarks/test_placeholder.py::test_import_time 6338.331177465948 iter/sec (stddev: 0.000035300431259638925) 7365.802725990653 iter/sec (stddev: 0.00000874050089396922) 1.16

This comment was automatically generated by workflow using github-action-benchmark.

@endavis endavis added the ready-to-merge PR is reviewed and ready to merge label Apr 3, 2026
@endavis endavis merged commit 2a4d52e into main Apr 3, 2026
16 of 17 checks passed
@endavis endavis deleted the chore/482-sync-pyproject-template branch April 3, 2026 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge PR is reviewed and ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant