fix(#42): CI — markdown lint auto-fix and lycheeignore for private repos#44
Conversation
- Run markdownlint-cli2 --fix on all .md files (MD022, MD032 blank-line violations) - Add .lycheeignore to exclude private portfolio repo links from unauthenticated checks Closes #42 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
atlas-apex
left a comment
There was a problem hiding this comment.
Code Review: PR #44
Commit: 4164825cbe9c6473bb3f8281cc65b45e209fa7d6
Summary
This PR auto-fixes MD022/MD032 markdown lint violations (blank lines around headings and lists) across 40 files, and adds a .lycheeignore file to suppress false-positive link check failures for private/portfolio GitHub repos. All changes are additive (173 additions, 0 deletions) — every change is a blank line insertion to satisfy markdownlint rules.
Checklist Results
- N/A Architecture & Design: No code changes — markdown docs only
- PASS Code Quality: Auto-fixer output is consistent and correct
- N/A Testing: No testable code changed
- PASS Security: No secrets, no code, no injection surface
- N/A Performance: Markdown-only changes
- PASS PR Description & Glossary: Summary, test plan, and glossary all present. Terms (MD022, MD032, .lycheeignore) defined clearly.
- N/A Technical Decisions (AgDR): No technical decisions — this is a lint auto-fix pass
Issues Found
None blocking.
Suggestions (non-blocking)
-
templates/prd.md— empty blockquote lines: The auto-fixer added bare>lines between headings and blockquote content in the User Stories section (lines 43 and 55 in the patched file). This renders as an empty blockquote paragraph above the user story text. It satisfies MD022 but is slightly odd semantically. Consider replacing the bare>with just a blank line if markdownlint can be satisfied another way — or leave it as-is since it is cosmetically minor. -
.lycheeignore— broad patterns: The regexhttps://github.com/me2resh/apexstack.*will match any URL that starts with that prefix, including hypothetical subpaths of other repos that share the prefix. This is fine today but worth noting if the org adds repos with overlapping name prefixes. -
handover/SKILL.md— blank lines inside fenced code blocks: The auto-fixer added blank lines inside fenced code blocks (around the "Ready to add {name}" prompt). These are harmless but add empty lines in the rendered code block output. Non-issue.
Verdict
APPROVED — Clean, mechanical lint fix. All CI checks passing (markdownlint, lychee, shellcheck, PR title). No functional changes, no security concerns, no technical decisions requiring AgDR.
Reviewed by Rex (Code Reviewer Agent)
Reviewed commit: 4164825cbe9c6473bb3f8281cc65b45e209fa7d6
fix(me2resh#42): CI — markdown lint auto-fix and lycheeignore for private repos
fix(me2resh#42): CI — markdown lint auto-fix and lycheeignore for private repos
Summary
markdownlint-cli2 --fixacross all .md files — fixes MD022/MD032 blank-line violations in 40 files.lycheeignoreto exclude private portfolio repo links from unauthenticated link checksTest plan
Glossary
Generated with Claude Code