fix: ground-up lint cleanup — 181 violations → 0, merge redundant code-review skill (#137)

Markdownlint cleanup across 69 files (formatting only, no content changes):
- MD032 blanks-around-lists, MD013 line-length, MD031 blanks-around-fences
- MD040 code-fence-language, MD022 blanks-after-headings, MD028 blockquote-blanks
- Added .markdownlint.jsonc project config and updated .markdownlintignore

feature-dev 1.1.0 → 1.2.0:
- Merged redundant code-review skill into /review command (95% identical)
- Severity table, checklist, and vulnerability patterns now in single entry point
- Deleted skills/code-review/ directory

Validation: PASSED 0 failures, 0 warnings (first fully clean run)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: ANcpLua

.claude-plugin/marketplace.json

@@ -5,7 +5,7 @@
     "url": "https://github.com/ANcpLua"
   },
   "metadata": {
-    "description": "Claude Code plugin marketplace: 8 plugins, 23 commands, 6 skills, 14 agents. Multi-agent orchestration, cognitive amplification, OpenTelemetry docs, and .NET build enforcement."
+    "description": "Claude Code plugin marketplace: 8 plugins, 23 commands, 5 skills, 14 agents. Multi-agent orchestration, cognitive amplification, OpenTelemetry docs, and .NET build enforcement."
   },
   "plugins": [
     {
@@ -35,7 +35,7 @@
     {
       "name": "feature-dev",
       "description": "Guided feature development with specialized agents for codebase exploration, architecture design, code review, and quality assurance.",
-      "version": "1.1.0",
+      "version": "1.2.0",
       "source": "./plugins/feature-dev"
     },
     {

.markdownlint.jsonc

@@ -0,0 +1,38 @@
+{
+  // ancplua-claude-plugins markdownlint configuration
+  "default": true,
+
+  // MD013: Line length — 120 for our files, disabled for CHANGELOG (long entries are fine)
+  "MD013": {
+    "line_length": 120,
+    "heading_line_length": 120,
+    "code_block_line_length": 200,
+    "tables": false
+  },
+
+  // MD024: Duplicate headings — allowed in CHANGELOG (### Added, ### Fixed per version)
+  "MD024": {
+    "siblings_only": true
+  },
+
+  // MD004: Unordered list style — dashes
+  "MD004": {
+    "style": "dash"
+  },
+
+  // MD029: Ordered list prefix — ordered (1/2/3)
+  "MD029": {
+    "style": "ordered"
+  },
+
+  // MD050: Strong style — asterisks
+  "MD050": {
+    "style": "asterisk"
+  },
+
+  // MD060: Table column style — disabled (too noisy for our mixed table formats)
+  "MD060": false,
+
+  // MD041: First line heading — disabled (hookify .local.md files have YAML frontmatter)
+  "MD041": false
+}

.markdownlintignore

@@ -4,5 +4,8 @@ plugins/otelwiki/docs/
 # Append-only history — reformatting past entries has no value
 CHANGELOG.md
 
+# Blog post content
+Claude Code Agent Teams*.md
+
 # Node modules
 node_modules/

CHANGELOG.md

@@ -8,6 +8,23 @@ and the project follows [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 
 ### Fixed
 
+- **markdownlint**: Ground-up cleanup — 181 violations → 0 across 69 files. MD032 (blanks-around-lists), MD013 (line-length), MD031 (blanks-around-fences), MD040 (code-fence-language), MD022 (blanks-after-headings), MD028 (blockquote-blanks), MD029 (ordered-list-prefix). Formatting only, no content changes
+
+### Added
+
+- **`.markdownlint.jsonc`**: Project-wide markdownlint configuration — line length 120, dashes for lists, asterisks for bold, MD060/MD041 disabled
+- **`.markdownlintignore`**: Added blog post exclusion pattern
+
+### Removed
+
+- **`feature-dev/skills/code-review`**: Deleted redundant skill — 95% identical to `/review` command. Merged severity table, checklist, and common vulnerability patterns into `/review` command. One entry point instead of two
+
+### Changed
+
+- **`feature-dev`**: Bumped 1.1.0 → 1.2.0. `/review` command now contains the full 6-step workflow, severity levels table, checklist, and common vulnerability patterns previously split across the skill and command
+
+### Fixed
+
 - **`hookify/rule_engine.py`**: Fixed PreToolUse deny putting message in `systemMessage` (user-only display) instead of `permissionDecisionReason` (the field Claude actually receives). Root cause of Claude never seeing hook guidance when tools are blocked. Also split PreToolUse/PostToolUse into separate branches with correct response formats per Claude Code hooks spec
 - **`hookify/hook_runner.py`**: Fixed error reporting — hookify crashes now use exit code 2 + stderr (Claude-visible) instead of `systemMessage` (user-only)
 - **`hookify/global-rules/mtp-smart-test-filtering`**: Rewrote rule message to be directive — shorter (8 lines vs 80) with explicit "pick one and retry" action items

CLAUDE.md

@@ -104,6 +104,7 @@ but NOT conversation history. All context must be in the spawn prompt.
 See `docs/ARCHITECTURE.md` for the complete repository layout, plugin structure, and ecosystem diagram.
 
 Key paths:
+
 - Plugins: `plugins/<name>/` — 8 plugins (23 commands, 6 skills, 14 agents)
 - Rules (auto-loaded): `.claude/rules/`
 - Plugin registry: `.claude-plugin/marketplace.json`
@@ -595,7 +596,8 @@ You MUST NOT:
 
 ## 8. CI & Validation
 
-See `docs/WORKFLOWS.md` for complete CI pipeline documentation, exact shell commands, and the cross-repo `trigger-docs.yml` workflow.
+See `docs/WORKFLOWS.md` for complete CI pipeline documentation, exact shell commands, and the
+cross-repo `trigger-docs.yml` workflow.
 
 ### Local Validation
 

docs/ARCHITECTURE.md

@@ -108,7 +108,8 @@ Plugins contribute to the agent's passive context via three layers (in load orde
 2. **`SKILL.md` frontmatter `description`** — loaded when skill is referenced
 3. **`hooks/hooks.json` SessionStart → `additionalContext`** — dynamic, computed at session start
 
-Use SessionStart hooks when context must be computed at runtime (version detection, prior findings, project-specific routing). Use CLAUDE.md for stable rules. Use skill descriptions for task-routing cues.
+Use SessionStart hooks when context must be computed at runtime (version detection, prior findings,
+project-specific routing). Use CLAUDE.md for stable rules. Use skill descriptions for task-routing cues.
 
 ---
 
@@ -130,7 +131,8 @@ CI mirrors the same checks via `.github/workflows/ci.yml`.
 
 ## 5. Design principles (SOLID for plugins)
 
-See `.claude/rules/solid-principles.md` (auto-loaded at session start) for the full SOLID plugin design constraints including the plugin responsibility table.
+See `.claude/rules/solid-principles.md` (auto-loaded at session start) for the full SOLID plugin design
+constraints including the plugin responsibility table.
 
 ---