Skip to content

🔧 Add AGENTS.md and copilot-setup-steps workflow#132

Open
Copilot wants to merge 3 commits intomasterfrom
copilot/fix-parser-bug
Open

🔧 Add AGENTS.md and copilot-setup-steps workflow#132
Copilot wants to merge 3 commits intomasterfrom
copilot/fix-parser-bug

Conversation

Copy link

Copilot AI commented Feb 28, 2026

Mirror of executablebooks/markdown-it-py#380 for this repo.

  • AGENTS.md: AI coding agent guide covering plugin architecture (block/inline/core rule patterns), development commands (tox), code style (Ruff/Mypy strict), fixture-based test patterns, and common operations (adding plugins, rules, renderers)
  • .github/workflows/copilot-setup-steps.yml: Copilot environment setup — Python 3.11, uv, pre-commit, tox
  • tox.ini: Added pre-commit and mypy tox environments so all commands referenced in AGENTS.md are available; fixed docs builder command to use {posargs} correctly (tox -e docs-update -- linkcheck instead of BUILDER=linkcheck)
Original prompt

Can you do similar to executablebooks/markdown-it-py#380 here


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: chrisjsewell <2997570+chrisjsewell@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix parser bug in markdown-it-py Add AGENTS.md and copilot-setup-steps workflow Feb 28, 2026
@codecov
Copy link

codecov bot commented Feb 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.96%. Comparing base (d11bdaf) to head (a664c70).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
+ Coverage   92.80%   92.96%   +0.16%     
==========================================
  Files          31       32       +1     
  Lines        1835     1906      +71     
==========================================
+ Hits         1703     1772      +69     
- Misses        132      134       +2     
Flag Coverage Δ
pytests 92.96% <ø> (+0.16%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@chrisjsewell
Copy link
Member

@copilot can you make sure that all the tox commands specified in the agents.md are actually available in the tox configuration, and if not add them (if applicable)

Also generally review the agents.md

Co-authored-by: chrisjsewell <2997570+chrisjsewell@users.noreply.github.com>
Copy link
Author

Copilot AI commented Feb 28, 2026

@copilot can you make sure that all the tox commands specified in the agents.md are actually available in the tox configuration, and if not add them (if applicable)

Also generally review the agents.m...

Audited all tox commands in AGENTS.md against tox.ini in a664c70:

  1. Fixed incorrect command: BUILDER=linkcheck tox -e docs-updatetox -e docs-update -- linkcheck (tox.ini uses {posargs:html}, not a $BUILDER env var)
  2. Added missing tox environments to tox.ini:
    • pre-commit — runs pre-commit run {posargs:--all-files}
    • mypy — runs mypy mdit_py_plugins {posargs}
  3. Updated AGENTS.md Code Quality section with the new tox commands

Verified all environments work: tox -e mypy passes (32 source files, no issues), tox -e pre-commit passes (all hooks green), tox -e py310 passes (436 tests).

@chrisjsewell chrisjsewell marked this pull request as ready for review February 28, 2026 13:27
@chrisjsewell chrisjsewell changed the title Add AGENTS.md and copilot-setup-steps workflow 🔧 Add AGENTS.md and copilot-setup-steps workflow Feb 28, 2026
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.

2 participants