Skip to content

Conversation

@HartBrook
Copy link
Owner

Summary

  • Rename command from /lookagain to /look:again (plugin name look, command again)
  • Add test suite (scripts/test.sh) covering plugin.json validation, required files, frontmatter, cross-references, settings, and build output
  • Add Makefile with build, test, dev, clean targets
  • Add CONTRIBUTING.md with development setup and workflow docs
  • Harden CI/CD: pin actions/checkout to SHA (v4.2.2), fix echo to printf in release notes
  • Fix scripts: wrap cd in subshell in package.sh, use sys.argv in test.sh to avoid injection, report all missing frontmatter fields, add python3 guard
  • Fix permission mismatch: align again.md to use Write tool (matching Write(.lookagain/**) setting)
  • Remove dev-install.sh: redundant with make dev
  • Improve reviewer agent: more direct instructions to read source code rather than exploring broadly

Test plan

  • make test passes (33/33)
  • make dev starts Claude Code with the plugin loaded
  • /look:again invokes the renamed command correctly
  • Release workflow still builds and zips correctly on tag push

- Pin actions/checkout to SHA (v4.2.2) in CI and release workflows
- Fix package.sh cd leak by wrapping zip step in subshell
- Fix test.sh: check_json_field uses sys.argv instead of string
  interpolation, check_frontmatter reports all missing fields,
  add python3 availability guard
- Align again.md to use Write tool matching the settings permission
- Remove redundant dev-install.sh (duplicated by make dev)
- Rename command from lookagain to look:again (plugin name: look)
- Add Makefile, CONTRIBUTING.md, and comprehensive test suite

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@HartBrook HartBrook merged commit 35b7f38 into main Jan 28, 2026
1 check passed
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