Merge pull request 'hotfix(data-platform): use separate hooks.json file' (#157 ) from hotfix/data-platform-hooks into main

Reviewed-on: #157
docs: correct plugin.json hooks format rules
2026-01-25 20:46:14 +00:00 · 2026-01-25 15:42:13 -05:00 · 2026-01-25 15:41:46 -05:00 · 2026-01-25 20:37:15 +00:00 · 2026-01-25 20:24:29 +00:00 · 2026-01-25 19:37:53 +00:00
3 changed files with 13 additions and 9 deletions
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -164,9 +164,11 @@ leo-claude-mktplace/
 - **CLI tools forbidden** - Use MCP tools exclusively (never `tea`, `gh`, etc.)

 #### ⚠️ plugin.json Format Rules (CRITICAL)
- **Hooks MUST be inline** - NEVER use `"hooks": "path/to/file.json"`
+- **Hooks in separate file** - Use `hooks/hooks.json` (auto-discovered), NOT inline in plugin.json
+- **NEVER reference hooks** - Don't add `"hooks": "..."` field to plugin.json at all
 - **Agents auto-discover** - NEVER add `"agents": ["./agents/"]` - .md files found automatically
 - **Always validate** - Run `./scripts/validate-marketplace.sh` before committing
+- **Working examples:** projman, pr-review, claude-config-maintainer all use `hooks/hooks.json`
 - See lesson: `lessons/patterns/plugin-manifest-validation---hooks-and-agents-format-requirements`

 ### Hooks (Valid Events Only)
--- a/plugins/data-platform/.claude-plugin/plugin.json
+++ b/plugins/data-platform/.claude-plugin/plugin.json
@@ -18,14 +18,6 @@
    "etl",
    "dataframe"
  ],
-  "hooks": {
-    "SessionStart": [
-      {
-        "type": "command",
-        "command": "${CLAUDE_PLUGIN_ROOT}/hooks/startup-check.sh"
-      }
-    ]
-  },
  "commands": ["./commands/"],
  "mcpServers": ["./.mcp.json"]
 }
--- a/plugins/data-platform/hooks/hooks.json
+++ b/plugins/data-platform/hooks/hooks.json
@@ -0,0 +1,10 @@
+{
+  "hooks": {
+    "SessionStart": [
+      {
+        "type": "command",
+        "command": "${CLAUDE_PLUGIN_ROOT}/hooks/startup-check.sh"
+      }
+    ]
+  }
+}
Author	SHA1	Message	Date
Leo Miranda	1df9573f7a	Merge pull request 'hotfix(data-platform): use separate hooks.json file' (#157 ) from hotfix/data-platform-hooks into main Reviewed-on: #157	2026-01-25 20:46:14 +00:00
lmiranda	35d5f14003	docs: correct plugin.json hooks format rules Hooks should be in separate hooks/hooks.json file (auto-discovered), NOT inline in plugin.json. Previous documentation was wrong. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>	2026-01-25 15:42:13 -05:00
lmiranda	5321b2929e	fix(data-platform): use separate hooks.json file (not inline) Previous fix was wrong. Hooks should be in separate hooks/hooks.json file (auto-discovered), NOT inline in plugin.json. Pattern matches working plugins: projman, pr-review, claude-config-maintainer Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>	2026-01-25 15:41:46 -05:00
Leo Miranda	05aa50d409	Merge pull request 'development' (#156 ) from development into main Reviewed-on: #156	2026-01-25 20:37:15 +00:00
Leo Miranda	54e8e694b1	Merge pull request 'development' (#154 ) from development into main Reviewed-on: #154	2026-01-25 20:24:29 +00:00
Leo Miranda	8ec7cbb1e9	Merge pull request 'development' (#152 ) from development into main Reviewed-on: #152	2026-01-25 19:37:53 +00:00
Leo Miranda	3519a96d06	Merge pull request 'development' (#150 ) from development into main Reviewed-on: #150	2026-01-25 19:28:43 +00:00
Leo Miranda	f809c672b5	Merge pull request 'development' (#148 ) from development into main Reviewed-on: #148	2026-01-24 18:16:29 +00:00
Leo Miranda	5d205c9c13	Merge pull request 'development' (#142 ) from development into main Reviewed-on: #142	2026-01-24 17:36:43 +00:00
Leo Miranda	efb83e0f28	Merge pull request 'development' (#135 ) from development into main Reviewed-on: #135	2026-01-24 16:48:40 +00:00
Leo Miranda	7bedfa2c65	Merge pull request 'development' (#133 ) from development into main Reviewed-on: #133	2026-01-23 22:54:42 +00:00
Leo Miranda	42ab4f13cf	Merge pull request 'development' (#125 ) from development into main Reviewed-on: #125	2026-01-23 21:48:13 +00:00
Leo Miranda	77dc122079	Merge pull request 'development' (#122 ) from development into main Reviewed-on: #122	2026-01-23 21:08:10 +00:00
Leo Miranda	ce774bcc6f	Merge pull request 'development' (#119 ) from development into main Reviewed-on: #119	2026-01-23 19:47:39 +00:00
Leo Miranda	b3abe863af	Merge pull request 'development' (#117 ) from development into main Reviewed-on: #117	2026-01-23 17:50:25 +00:00