chore: add AI tooling config and conventional commits convention (JDWLABS-21)

[jdwillmsen]

Summary

• Add CLAUDE.md with NX monorepo structure, dev commands, and AI agent contract
• Add GEMINI.md, AGENTS.md, .claude/settings.json, CONTRIBUTING.md, PR template
• Fix .husky/commit-msg shebang for Windows compatibility (pre-existing exec format error)

Jira

JDWLABS-21 — part of JDWLABS-14 (Org Health & Governance)

Test plan

• Verify CLAUDE.md accurately reflects repo structure (apps/angular/, apps/go/, apps/springboot/)
• Verify .claude/settings.json denies git push and rm -rf
• Verify commit-msg hook works: try a bad commit message, expect rejection

🤖 Generated with Claude Code

[jdwillmsen]

Code review

Found 3 issues:

1. .claude/settings.json allows git commit* which permits git commit --no-verify, bypassing the commitlint and lint-staged hooks. The existing AGENT.md on main explicitly lists git commit --no-verify under "Never Run". The entry should be scoped to prevent the --no-verify flag (e.g. split into "Bash(git commit -m *)" and "Bash(git commit --amend*)", or add a deny entry for "Bash(git commit --no-verify*)").

apps/.claude/settings.json

Line 13 in a76f312

"Bash(git commit*)"

2. The new CLAUDE.md uses npx nx throughout, but the repo's existing convention (in both CLAUDE.md and AGENT.md on main) is pnpm exec nx. Using npx nx can resolve a different version of nx than the one installed in the pnpm lockfile, and contradicts the explicit "Never run npm install or yarn" contract in AGENT.md. All npx nx invocations should be pnpm exec nx to match the existing convention.

apps/CLAUDE.md

Lines 31 to 57 in a76f312

	npx nx run-many -t build # Build all apps
	```
	### Test
	```bash
	npx nx test <app-name> # Unit tests for one app
	npx nx run-many -t test # All unit tests
	npx nx e2e <app-name>-e2e # Cypress E2E tests
	```
	### Lint
	```bash
	npx nx lint <app-name> # Lint one app
	npx nx run-many -t lint # Lint all
	```
	### Affected (used in CI to scope work to changed code)
	```bash
	npx nx affected -t build # Build only what changed vs main
	npx nx affected -t test # Test only what changed
	npx nx affected -t lint # Lint only what changed
	```
	### Go (backend services)

3. The PR adds .github/pull_request_template.md (lowercase filename) but .github/PULL_REQUEST_TEMPLATE.md (uppercase) already exists on main. On Linux (case-sensitive) filesystems these are two distinct files. After merge both files will exist, which is ambiguous. GitHub will use whichever it finds first; the intended template may not be picked up. The new file should either replace the existing one (delete the old) or use the same casing.

apps/.github/pull_request_template.md

Lines 1 to 24 in a76f312

	## What
	<!-- One sentence: what changed and why -->
	## Type of change
	- [ ] `feat` — new feature or capability
	- [ ] `fix` — bug fix
	- [ ] `build` — build system or external dependency change
	- [ ] `chore` — maintenance / config / tooling
	- [ ] `ci` — CI/CD pipeline change
	- [ ] `docs` — documentation only
	- [ ] `perf` — performance improvement
	- [ ] `refactor` — restructure, no behavior change
	- [ ] `revert` — revert a previous commit
	- [ ] `style` — formatting / whitespace (no logic change)
	- [ ] `test` — test additions or updates
	## Checklist
	- [ ] PR title follows conventional commit format: `type(scope): description`
	- [ ] `npx nx affected -t build,test,lint` passes locally
	- [ ] No secrets or credentials in diff
	- [ ] CONTRIBUTING.md consulted for commit message format

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}

[nx-cloud]

View your CI Pipeline Execution ↗ for commit e8873b4

Command	Status	Duration	Result
nx affected -t build	✅ Succeeded	<1s	View ↗
nx affected -t lint test	✅ Succeeded	<1s	View ↗
nx-cloud record -- nx format:check	✅ Succeeded	<1s	View ↗

---

☁️ Nx Cloud last updated this comment at 2026-06-03 05:53:37 UTC

[nx-cloud]

View your CI Pipeline Execution ↗ for commit 86a663a

Command	Status	Duration	Result
nx-cloud record -- nx format:check	❌ Failed	<1s	View ↗

---

☁️ Nx Cloud last updated this comment at 2026-06-03 04:29:59 UTC

[nx-cloud]

View your CI Pipeline Execution ↗ for commit 86a663a

Command	Status	Duration	Result
nx-cloud record -- nx format:check	❌ Failed	<1s	View ↗

---

☁️ Nx Cloud last updated this comment at 2026-06-03 04:29:59 UTC

[nx-cloud]

Nx Cloud is proposing a fix for your failed CI:

We ran nx format to fix the Prettier formatting failure on .claude/settings.json, which was introduced by this PR but not formatted before being committed. Prettier collapsed the short deny array onto a single line, which is the only change needed to satisfy nx format:check.

Tip

✅ We verified this fix by re-running nx-cloud record -- nx format:check.

diff --git a/.claude/settings.json b/.claude/settings.json
index 7a0bbf3..6023809 100644
--- a/.claude/settings.json
+++ b/.claude/settings.json
@@ -12,9 +12,6 @@
       "Bash(git add *)",
       "Bash(git commit*)"
     ],
-    "deny": [
-      "Bash(git push*)",
-      "Bash(rm -rf *)"
-    ]
+    "deny": ["Bash(git push*)", "Bash(rm -rf *)"]
   }
 }

  

Or Apply changes locally with:

npx nx-cloud apply-locally SxIm-YONs

Apply fix locally with your editor ↗   View interactive diff ↗

_{🎓 Learn more about Self-Healing CI on nx.dev}