Skip to content

fix(review): resolve PR #244 review threads, harden markdown fence/asterisk parsing, and update release notes (#265) - #266

Merged
dan-petty merged 2 commits into
release/v0.2.20from
fix/265-resolve-pr-244-review-threads
Sep 18, 2026
Merged

dan-petty merged 2 commits into
release/v0.2.20from
fix/265-resolve-pr-244-review-threads

Conversation

@dan-petty

Copy link
Copy Markdown
Owner

Closes #265

Summary of Changes

Remediates the 6 unresolved review discussion threads on release PR #244:

  • Line-Start Fence Parser Hardening: Replaced literal backtick counting with line-start delimiter inspection (balance_markdown_fences), properly handling inline backtick strings like print("```") by wrapping with 4-backtick code fences.
  • Heading & Table Asterisk Escaping: Escapes un-backticked asterisks across detailed finding headings and summary table rows to prevent Markdown formatting collisions with Python unpacking like **kwargs.
  • Paired Bold Stripping: Conditioned bold stripping on matched outer pairs, ensuring un-backticked inner bold delimiters are not present and preserving legitimate **kwargs prefixes.
  • Dynamic Version Interpolation in Generator Test: Replaced hardcoded version fixture with devops_cli.version.
  • Comprehensive Release Notes: Expanded v0.2.20 release notes in CHANGELOG.md with all shipped deliverables.
  • Review Threads Resolved: Replied in-thread to all 6 review discussion threads on PR feat(release): v0.2.20 #244 and marked them resolved.
  • Quality Gates: All 13 CI quality gates verified passing locally (3,395+ tests, coverage >= 90%, 0 lint/type errors).

…terisk parsing, and update release notes (#265)
@dan-petty dan-petty added this to the v0.2.20 milestone Sep 18, 2026
@dan-petty
dan-petty merged commit 12ac55a into release/v0.2.20 Sep 18, 2026
4 checks passed
@dan-petty
dan-petty deleted the fix/265-resolve-pr-244-review-threads branch September 18, 2026 23:24
dan-petty added a commit that referenced this pull request Sep 19, 2026
* feat(release): v0.2.20

* fix(pr): handle JSON array timeline payload in _parse_timeline_copilot_state (#245) (#246)

* fix(pr): handle JSON array timeline payload in _parse_timeline_copilot_state (#245)

* docs(tasks): update task 245 status to In Review and link PR #246

* docs(agents): add roadmap ingestion rules for missing parameters and bad patterns (#247)

* docs(agents): add roadmap ingestion rules for missing parameters and bad patterns

* fix(agents): gate devops roadmap governance block on is_devops_cli

* fix(review): remediate markdown formatting issues and unbalanced code blocks in review reports (#250) (#251)

* refactor(core): use pyproject.toml as single source of truth for version (#248) (#249)

* refactor(core): use pyproject.toml as single source of truth for version (#248)

* fix(release): require __version__ presence in init before dynamic fallback (#249)

* docs(roadmap): add universal subcommand option propagation task

* feat(cli): universal trailing --dry-run option propagation across subcommands (#252) (#253)

* feat(pm): forward-looking project management, roadmap reconstruction, compactor hardening & issue sync (#254) (#255)

* feat(pm): forward-looking project management, roadmap reconstruction, compactor hardening & issue sync (#254)

* docs(agent): link PR #255 to task-254

* fix(github): handle remote issue query failures gracefully in roadmap sync (#254)

* feat(ci): fast execution caching when unchanged & pre-commit file change tracking integration (#256) (#257)

* feat(ci): fast execution caching when unchanged & pre-commit file change tracking integration (#256)

* docs(agent): link PR #257 to task-256

* feat(pr): add integrations to keep pull requests up to date (#258) (#259)

* feat(pr): add integrations to keep pull requests up to date (#258)

* docs(agent): link PR #259 to task-258

* fix(ci): address actionlint shellcheck warning and add shellcheck to devcontainer

* docs(roadmap): prune completed items from Value vs. Effort Prioritization Matrix

* feat(ci): accelerate devops ci performance with worker auto-scaling, test hotspot isolation, and zero-blocking dispatch (#260) (#261)

* docs(roadmap): add v0.2.24 milestone for Vibes architectural alignment

- POSIX Process Group Sandbox Enforcement (Obs 05)
- Structural Pre-Commit Hook Inversion (Systems Obs 08)
- Anti-Brittle Constant Elimination (Obs 06)
- Semantic Validator Deprecation & Structural Positional Oracles (Obs 17)

* feat(ci): accelerate devops ci performance with worker auto-scaling, test hotspot isolation, and zero-blocking dispatch (#260)

* fix(review): remediate session findings, harden defensive boundaries, and improve review feedback loop (#262) (#263)

* fix(review): remediate session findings, harden defensive boundaries, and improve review feedback loop (#262)

* docs(agent): link PR #263 and update status to In Review

* docs(agent): reconcile task-262 as merged and prune completed item from matrix (#264)

* fix(review): resolve PR #244 review threads, harden markdown fence/asterisk parsing, and update release notes (#265) (#266)

* fix(review): resolve PR #244 review threads, harden markdown fence/asterisk parsing, and update release notes (#265)

* docs(agent): link PR #266 and set status to In Review

* refactor(core): replace inconsistent N-Gate references with Gated (#267) (#268)

* refactor(core): replace inconsistent N-Gate references with Gated (#267)

* docs(agent): link PR #268 in task-267 tracking

* docs(agent): author deliverables as merged and prohibit administrative tracking PRs (#270) (#271)
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