Skip to content

feat: harness-audit workflow + expanded language rules - #89

Merged
5uck1ess merged 3 commits into
mainfrom
feat/harness-audit-rules-expansion
Apr 13, 2026
Merged

feat: harness-audit workflow + expanded language rules#89
5uck1ess merged 3 commits into
mainfrom
feat/harness-audit-rules-expansion

Conversation

@5uck1ess

@5uck1ess 5uck1ess commented Apr 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • New harness-audit workflow + skill — audits the agent harness config itself (CLAUDE.md, hooks, MCP configs, agent definitions, permissions) for misconfigurations, injection risks, and security gaps. Inspired by ECC's AgentShield concept but engine-driven.
  • New common.md rules — cross-platform, language-agnostic rules (paths, temp dirs, line endings, assumption-surfacing, error messages). Applied via **/* glob to all files.
  • 4 new language rule sets — java, kotlin, swift, csharp. Follow existing pattern (YAML frontmatter with path globs, one-liner bullets).
  • CLAUDE.md convention — added "state assumptions before implementing" to Conventions section. Updated layout table to reflect expanded rules.

Motivated by comparison with everything-claude-code — took the ideas worth adapting to our deterministic architecture, left the rest.

Changelog

Added

  • harness-audit workflow + skill — audits agent harness configuration (CLAUDE.md, hooks, MCP configs, agent definitions, permissions) for misconfigurations, injection risks, and security gaps
  • common.md language-agnostic coding rules — cross-platform paths, temp dirs, assumption-surfacing, error messages (applied via **/* glob)
  • java, kotlin, swift, csharp language rule sets — following existing pattern with path-gated activation

Changed

  • CLAUDE.md: added "state assumptions before implementing" convention, expanded rules listing in layout table
  • README.md: added 6 new entries to coding rules table, bumped skill count to 39
  • ROADMAP.md: bumped to 39 skills / 22 workflows

Test plan

  • devkit workflow list shows harness-audit
  • /devkit:harness-audit runs through all 3 steps (inventory → analyze → report)
  • /devkit:setup-rules installs all 10 rule files to ~/.claude/rules/
  • Existing workflows unaffected (no changes to engine or existing YAML)
  • Rule paths: globs activate correctly per language
  • No Windows-breaking shell commands in new files (all prompt-driven, no test -f etc.)

…ules

- New harness-audit workflow + skill: audits CLAUDE.md, hooks, MCP configs,
  agent definitions, and permissions for misconfigurations and injection risks
- New common.md rules: cross-platform, language-agnostic (paths, temp dirs,
  assumption-surfacing, error messages)
- New language rules: java, kotlin, swift, csharp
- CLAUDE.md: add "state assumptions" convention, update layout table
- README: add common/java/kotlin/swift/csharp to coding rules table, bump skill count to 39
- ROADMAP: bump to 39 skills, 22 workflows, add harness-audit to lists
CI validate-counts caught two more spots in README.md and one in
ROADMAP.md still saying 21 workflows.
@5uck1ess
5uck1ess merged commit 9a9cdba into main Apr 13, 2026
6 checks passed
@5uck1ess
5uck1ess deleted the feat/harness-audit-rules-expansion branch April 13, 2026 16:13
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