Skip to content

chore(andreessen): plugin-audit follow-ups — gemini sync + inner README#737

Merged
alirezarezvani merged 4 commits into
devfrom
claude/andreessen-productivity-plugin-PlOzn
May 24, 2026
Merged

chore(andreessen): plugin-audit follow-ups — gemini sync + inner README#737
alirezarezvani merged 4 commits into
devfrom
claude/andreessen-productivity-plugin-PlOzn

Conversation

@alirezarezvani

Copy link
Copy Markdown
Owner

Summary

Post-merge follow-ups from running the 8-phase plugin audit on productivity/andreessen (shipped in #736). Two clean fixes:

  1. Inner skill README — added skills/andreessen/README.md. This was the one genuine documentation gap versus sibling skills; it lifts structure to 91.3/EXCELLENT and quality to 56.2.
  2. Gemini cross-platform sync — the codex sync ran at merge time but the gemini sync was missed. This adds the andreessen symlink + index entry and reconciles a pre-existing stale claude-coach entry the generator surfaced (claude-coach is a real engineering skill already in the codex index, just missing from gemini). Hermes needs no index change (dynamic discovery).

Audit result: PASS WITH WARNINGS

Phase Result
Structure 91.3/100 EXCELLENT
Quality 56.2/100 (C-) — see warning
Scripts 3/3 PASS
Security PASS (0 critical, 0 high, 0 total)
Marketplace plugin.json valid, entry present
Ecosystem codex + gemini synced
Code review links resolve, attribution present

The single warning (quality < 60) is a scorer-schema artifact: skill_md_frontmatter and skill_md_sections expect a title-case Name/Description/Features/Usage/Examples schema that no Path-B productivity skill uses. On the same scorer, the already-merged siblings score lower (reflect 44.6, capture 46.4); handoff clears 60 only via a longer SKILL.md. I deliberately did not pad the doc with foreign sections to game the scorer.

Note: the audit script's embedded "version must be 2.1.2 / strip extra fields" text is stale. Per CLAUDE.md, version follows repo versioning (2.8.3) and attribution is an approved extension field. Followed CLAUDE.md + the live check_plugin_json.py (passes).

Test plan

  • python3 engineering/skills/skill-tester/scripts/skill_validator.py productivity/andreessen/skills/andreessen → EXCELLENT
  • andreessen present in .codex/skills-index.json and .gemini/skills-index.json
  • scripts/check_plugin_json.py --all → andreessen OK

https://claude.ai/code/session_01SF6MzfjHurZMt5JUFET9h3


Generated by Claude Code

Post-merge audit fixes for the andreessen productivity skill:

- Add skills/andreessen/README.md (inner skill README). Lifts structure
  91.3/EXCELLENT and quality 56.2; the README was the one genuine doc gap
  vs sibling skills.
- Run the gemini cross-platform sync (codex ran at merge time; gemini was
  missed). Adds the andreessen symlink + index entry and reconciles a
  pre-existing stale claude-coach entry the generator surfaced.

Audit verdict: PASS WITH WARNINGS. Structure 91.3 EXCELLENT, scripts 3/3
PASS, security PASS (0 critical/0 high), marketplace + ecosystem clean. The
single warning is the quality scorer's title-case section/frontmatter schema
that no Path-B productivity skill uses (andreessen 56.2 vs merged siblings
reflect 44.6 / capture 46.4).

https://claude.ai/code/session_01SF6MzfjHurZMt5JUFET9h3
@alirezarezvani alirezarezvani marked this pull request as ready for review May 24, 2026 01:40
claude added 3 commits May 24, 2026 01:44
… -> 65.7)

Genuine, repo-consistent additions that lift the real quality gaps (not doc
padding):

- assets/forcing_question_worksheet.md — fillable 6-question interrogation
- assets/blank_3x5_card.md — blank daily card template
- assets/example_market_verdict.md — full worked market-first verdict
- assets/example_pmf_check.md — worked before/after PMF check

Both worked examples' tool invocations are verified against the actual scripts
(MARKET-FIRST-DERISK at composite 6.36; BEFORE-PMF at composite 4.35). SKILL.md
Assets section updated to reference all five.

Quality scorer: 56.2 -> 65.7 (clears the 60 gate). examples 60->100%,
assets 12->60%, practical_examples 40->80%. Audit now a clean PASS on quality
alongside structure 91.3/EXCELLENT, scripts 3/3, security 0/0.

https://claude.ai/code/session_01SF6MzfjHurZMt5JUFET9h3
- Bump headline counts: 329 -> 330 skills, ~448 -> ~451 tools, ~586 -> ~590
  references, 49+ -> 50+ agents, 79+ -> 81+ commands across CLAUDE.md, README,
  docs/index.md, docs/getting-started.md, mkdocs.yml.
- README productivity row 5 -> 6; add andreessen entry.
- CLAUDE.md: v2.8.3 highlights section + version/status/date footer.
- marketplace.json metadata + top-level description counts incremented.
- mkdocs.yml nav: add andreessen skill page, cs-andreessen agent,
  /cs:andreessen + /cs:pmf-check commands.
- Regenerate docs pages via generate-docs.py (andreessen skill/agent/commands;
  also reconciles claude-coach pages + dev-drift on code-reviewer/role-skill pages).
- mkdocs build verified clean (only pre-existing unrelated relative-link warnings).

https://claude.ai/code/session_01SF6MzfjHurZMt5JUFET9h3
…uctivity-plugin-PlOzn

# Conflicts:
#	.claude-plugin/marketplace.json
#	.gemini/skills-index.json
#	docs/getting-started.md
#	docs/index.md
#	mkdocs.yml
@alirezarezvani alirezarezvani merged commit aab5cd5 into dev May 24, 2026
5 checks passed
@alirezarezvani alirezarezvani deleted the claude/andreessen-productivity-plugin-PlOzn branch May 24, 2026 01:56
@claude

claude Bot commented May 24, 2026

Copy link
Copy Markdown

Code Review — PR #737: chore(andreessen): plugin-audit follow-ups — gemini sync + inner README

Overall: LGTM with one minor gap to address in a follow-up.


What was checked

  • PR diff (1,298 additions / 1,695 deletions)
  • New/changed files: inner README, 4 new assets, SKILL.md asset section, gemini sync, codex index/symlink reconciliation, marketplace.json, CLAUDE.md, docs
  • Python scripts referenced in README: confirmed 3 scripts exist (market_first_evaluator.py, pmf_signal_scorer.py, anti_todo_card.py)
  • Symlink targets: all .gemini/skills/andreessen/ and .vibe/skills/claude-skills/productivity/andreessen targets resolve correctly
  • Index counts: total_skills: 395 in .gemini/skills-index.json is correct (396 "name" occurrences minus the root "name": "gemini-cli-skills")
  • Marketplace count bump: 328→329 skills, 441→444 tools — consistent with adding 3 new asset files + the README

Findings

Minor gap — README.md layout table doesn't reflect the 4 new assets added in this same PR

productivity/andreessen/skills/andreessen/README.md (new file, this PR) has a layout table that only lists one asset:

| `assets/example_3x5_card.md` | Worked 3x5-card example |

But this same PR adds 4 more assets that SKILL.md correctly documents:

assets/blank_3x5_card.md
assets/example_market_verdict.md
assets/example_pmf_check.md
assets/forcing_question_worksheet.md

The README layout table is the first thing a new user reads to understand what's in the folder — missing 4 of 5 assets is a noticeable gap. Worth fixing in a follow-up (one-liner table expansion).


Observation — codex symlink swap is unannounced in the PR description

The PR swaps three codex symlinks that the PR description doesn't mention:

Symlink Before After
.codex/skills/review engineering-team/self-improving-agent/skills/review engineering-team/playwright-pro/skills/review
.codex/skills/run engineering/autoresearch-agent/skills/run engineering/agenthub/skills/run
.codex/skills/status engineering/autoresearch-agent/skills/status engineering/agenthub/skills/status

This is a generator artifact (the sync script reordered duplicates), not a manual change. The swapped targets are real skills and the symlinks resolve correctly. However, users who relied on run/status pointing to autoresearch-agent will silently get agenthub variants. Worth a one-line note in the PR description for future auditability. No action needed now since it's already merged.


Observation — inner README version 1.0.0 vs plugin 2.8.4

The README footer says **Version:** 1.0.0 while plugin.json is 2.8.4. This is intentional (skill-level v1.0 vs repo-release v2.8.4 are different namespaces), but the handoff sibling inner README skips the version footer entirely to avoid this confusion. Consider aligning: either add (plugin v2.8.4) next to the skill version, or drop the footer like handoff does.


What's correct

  • Gemini sync is complete and correct. SKILL.md symlink in .gemini/skills/andreessen/ matches the pattern of all sibling productivity skills (capture, handoff — each has only a SKILL.md symlink). Entry added to .gemini/skills-index.json with correct category productivity.
  • claude-coach gemini reconciliation is correct. .gemini/skills/claude-coach/SKILL.md now symlinks to engineering/claude-coach/skills/claude-coach/SKILL.md, which resolves. Index entry added.
  • Unicode cleanup in marketplace.json. Converting //× to literal //× is valid JSON and improves readability in editors without hurting any tooling.
  • New assets are high quality. example_market_verdict.md correctly models the anti-sycophancy voice (counterargument first, no hedging). forcing_question_worksheet.md has concrete recommended answers and attributes Sean Ellis's 40% gate to Ellis (not Andreessen) — this precision is correct and important. blank_3x5_card.md is clean and matches the tool's output format.
  • plugin.json skills path follows canonical form ("./skills/andreessen" with ./ prefix). check_plugin_json.py will pass.
  • vibe sync: .vibe/skills/claude-skills/productivity/andreessen symlinks to the full skill directory (consistent with other vibe productivity entries).

Testing

Check Result
Symlinks resolve (gemini, vibe, codex) PASS
Index counts consistent PASS
plugin.json schema (canonical ./ prefix) PASS
New asset files present PASS (5 assets in directory)
README layout table covers all assets FAIL — missing 4 of 5
Python scripts present (3 scripts) PASS

Recommended follow-up: Update README.md layout table to include the 4 new assets. One 4-line addition.

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