Skip to content

Improve codebase-readiness skill from skill-creator review#18

Merged
dgalarza merged 2 commits intomainfrom
improve-codebase-readiness-skill
Mar 11, 2026
Merged

Improve codebase-readiness skill from skill-creator review#18
dgalarza merged 2 commits intomainfrom
improve-codebase-readiness-skill

Conversation

@dgalarza
Copy link
Owner

Summary

  • Fix scoring bug: Phase 4 report template had hardcoded weights that didn't match either the dynamic or static weight tables from Phase 3. Report now uses placeholders with explicit instruction to apply the correct language-tier weights.
  • Extract recon script: Moved 35 lines of inline bash commands into scripts/recon.sh — more reliable, token-efficient, and fixes false-negative output for CI/CD, linting, and README detection.
  • Move report template to assets: 130-line markdown template extracted to assets/report-template.md, keeping SKILL.md focused on procedural instructions.
  • Add unsupported language fallback: Skill now gracefully handles codebases in Rust, C#, Kotlin, etc. instead of failing silently.
  • Writing style + description cleanup: Converted to imperative form per skill-creator guidelines, tightened frontmatter description.
  • Remove authoring notes: Deleted "thinner reference file with room for expansion" meta-notes from Go and Java language files.
  • Agent language scoping: Each agent prompt now instructs to reference only relevant sections of the language file.

Test plan

  • Packaging validation passes (package_skill.py)
  • Recon script tested against CreatorSignal (Ruby/Rails)
  • Recon script tested against tracewell.ai (Ruby + JS multi-language)
  • Full end-to-end assessment on a real codebase to verify weight tables apply correctly
  • Test unsupported language fallback with a Rust or C# project

- Fix weight table mismatch between Phase 3 scoring and Phase 4 report
  template (report now uses dynamic placeholders matching language tier)
- Extract inline bash into scripts/recon.sh for reliability
- Move report template to assets/report-template.md
- Add unsupported language fallback with graceful handling
- Convert writing style to imperative form throughout SKILL.md
- Tighten frontmatter description
- Remove authoring meta-notes from Go and Java language files
- Add per-agent language file scoping instructions
- Bump plugin version to 1.4.0
- Validate changed skills using quick_validate.py from skill-creator
- Validate marketplace.json and plugin.json for valid JSON syntax
- Two parallel jobs: validate-skills and validate-json
- Commit skill-creator to .agents/ for CI access
@dgalarza dgalarza force-pushed the improve-codebase-readiness-skill branch from 9c81f73 to 1b2269f Compare March 11, 2026 03:12
@dgalarza dgalarza merged commit f95af63 into main Mar 11, 2026
2 checks passed
@dgalarza dgalarza deleted the improve-codebase-readiness-skill branch March 11, 2026 03:14
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