update adding claude.md

Author: detain

.claude/hooks/caliber-check-sync.sh

@@ -0,0 +1,11 @@
+#!/bin/sh
+if grep -q "caliber" .git/hooks/pre-commit 2>/dev/null; then
+  exit 0
+fi
+FLAG="/tmp/caliber-nudge-$(echo "$PWD" | shasum | cut -c1-8)"
+find /tmp -maxdepth 1 -name "caliber-nudge-*" -mmin +120 -delete 2>/dev/null
+if [ -f "$FLAG" ]; then
+  exit 0
+fi
+touch "$FLAG"
+printf '{"decision":"block","reason":"Caliber agent config sync is not set up on this machine. Tell the user: This project uses Caliber to keep agent configs in sync — it is not set up yet. Want me to run /setup-caliber? (~30 seconds)"}'

.claude/settings.json

@@ -0,0 +1,79 @@
+{
+  "hooks": {
+    "Stop": [
+      {
+        "matcher": "",
+        "hooks": [
+          {
+            "type": "command",
+            "command": ".claude/hooks/caliber-check-sync.sh",
+            "description": "Caliber: offer setup if not configured"
+          }
+        ]
+      }
+    ],
+    "PostToolUse": [
+      {
+        "matcher": "",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "caliber learn observe",
+            "description": "Caliber: recording tool usage for session learning"
+          }
+        ]
+      }
+    ],
+    "PostToolUseFailure": [
+      {
+        "matcher": "",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "caliber learn observe --failure",
+            "description": "Caliber: recording tool failure for session learning"
+          }
+        ]
+      }
+    ],
+    "UserPromptSubmit": [
+      {
+        "matcher": "",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "caliber learn observe --prompt",
+            "description": "Caliber: recording user prompt for correction detection"
+          }
+        ]
+      }
+    ],
+    "SessionEnd": [
+      {
+        "matcher": "",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "caliber learn finalize --auto",
+            "description": "Caliber: finalizing session learnings"
+          }
+        ]
+      },
+      {
+        "matcher": "",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "caliber refresh --quiet",
+            "description": "Caliber: auto-refreshing docs based on code changes"
+          }
+        ]
+      }
+    ]
+  },
+  "permissions": {
+    "allow": [
+      "Bash(git *)"
+    ]
+  }
+}

.claude/skills/addon-lifecycle/SKILL.md

@@ -0,0 +1,106 @@
+---
+name: addon-lifecycle
+description: Implements doEnable/doDisable lifecycle methods following the pattern in src/Plugin.php. Handles get_module_settings(), myadmin_log(), function_requirements(), vps_get_password(), run_event('parse_service_extra'), and admin email via \MyAdmin\Mail()->adminMail(). Use when user says 'add enable logic', 'implement disable', 'provision addon', or works on activation/deactivation. Do NOT use for hook registration or AddonHandler setup.
+---
+# Addon Lifecycle
+
+## Critical
+
+- Always call `require_once __DIR__.'/../../../../include/licenses/license.functions.inc.php'` **before** any license or activation functions in both `doEnable` and `doDisable`.
+- Always resolve the service ID and customer ID through `$settings['PREFIX']` — never hardcode column names.
+- Call `myadmin_log()` immediately after `get_module_settings()` as the first observable action in both methods.
+- All external functions (`activate_*`, `deactivate_*`, `directadmin_get_best_type`) must be loaded via `function_requirements()`, not `require_once`.
+
+## Instructions
+
+1. **Declare the method signature** — both lifecycle methods share the same signature:
+   ```php
+   public static function doEnable(\ServiceHandler $serviceOrder, $repeatInvoiceId, $regexMatch = false)
+   public static function doDisable(\ServiceHandler $serviceOrder, $repeatInvoiceId, $regexMatch = false)
+   ```
+   Verify `$serviceOrder` is typed as `\ServiceHandler` before proceeding.
+
+2. **Extract service info and settings** — always the first two lines of the method body:
+   ```php
+   $serviceInfo = $serviceOrder->getServiceInfo();
+   $settings = get_module_settings(self::$module);
+   ```
+   All subsequent column references use `$settings['PREFIX']` as the key prefix.
+
+3. **Load license functions** — immediately after step 2, before any other calls:
+   ```php
+   require_once __DIR__.'/../../../../include/licenses/license.functions.inc.php';
+   ```
+
+4. **Log the lifecycle event:**
+   ```php
+   myadmin_log(self::$module, 'info', self::$name.' Activation', __LINE__, __FILE__, self::$module, $serviceInfo[$settings['PREFIX'].'_id']);
+   // or '... Deactivation' for doDisable
+   ```
+   Use `self::$name` (not a string literal) as the message prefix.
+
+5. **`doEnable` only — fetch password and parse extra:**
+   ```php
+   $pass = vps_get_password($serviceInfo[$settings['PREFIX'].'_id'], $serviceInfo[$settings['PREFIX'].'_custid']);
+   function_requirements('directadmin_get_best_type');
+   function_requirements('activate_directadmin'); // replace with your activation function
+   $serviceExtra = run_event('parse_service_extra', $serviceInfo[$settings['PREFIX'].'_extra'], self::$module);
+   $ostype = directadmin_get_best_type(self::$module, $serviceInfo[$settings['PREFIX'].'_type'], $serviceInfo, $serviceExtra);
+   ```
+   Verify `$pass` and `$ostype` are populated before calling the activation function.
+
+6. **`doEnable` only — call the provisioning function:**
+   ```php
+   activate_directadmin(
+       $serviceInfo[$settings['PREFIX'].'_ip'],
+       $ostype,
+       $pass,
+       $GLOBALS['tf']->accounts->cross_reference($serviceInfo[$settings['PREFIX'].'_custid']),
+       self::$module.$serviceInfo[$settings['PREFIX'].'_id']
+   );
+   ```
+
+7. **`doDisable` only — call the deprovisioning function then send admin email:**
+   ```php
+   function_requirements('deactivate_directadmin'); // replace with your deactivation function
+   deactivate_directadmin($serviceInfo[$settings['PREFIX'].'_ip']);
+   $email = $settings['TBLNAME'].' ID: '.$serviceInfo[$settings['PREFIX'].'_id'].'<br>'
+       .$settings['TBLNAME'].' Hostname: '.$serviceInfo[$settings['PREFIX'].'_hostname'].'<br>'
+       .'Repeat Invoice: '.$repeatInvoiceId.'<br>'
+       .'Description: '.self::$name.'<br>';
+   $subject = $settings['TBLNAME'].' '.$serviceInfo[$settings['PREFIX'].'_id'].' Canceled '.self::$name;
+   (new \MyAdmin\Mail())->adminMail($subject, $email, false, 'admin/vps_da_canceled.tpl');
+   ```
+   Verify the `.tpl` path exists under `include/templates/email/admin/` before committing.
+
+## Examples
+
+**User says:** "Implement doEnable and doDisable for a new Imunify addon"
+
+**Actions taken:**
+1. Add `public static $name = 'Imunify VPS Addon';` and `public static $module = 'vps';` to the Plugin class.
+2. Implement `doEnable` following steps 1–6, replacing `activate_directadmin` → `activate_imunify` and loading it via `function_requirements('activate_imunify')`.
+3. Implement `doDisable` following step 7, replacing `deactivate_directadmin` → `deactivate_imunify` and template `admin/vps_da_canceled.tpl` → `admin/vps_imunify_canceled.tpl`.
+
+**Result:**
+```php
+public static function doEnable(\ServiceHandler $serviceOrder, $repeatInvoiceId, $regexMatch = false)
+{
+    $serviceInfo = $serviceOrder->getServiceInfo();
+    $settings = get_module_settings(self::$module);
+    require_once __DIR__.'/../../../../include/licenses/license.functions.inc.php';
+    myadmin_log(self::$module, 'info', self::$name.' Activation', __LINE__, __FILE__, self::$module, $serviceInfo[$settings['PREFIX'].'_id']);
+    $pass = vps_get_password($serviceInfo[$settings['PREFIX'].'_id'], $serviceInfo[$settings['PREFIX'].'_custid']);
+    function_requirements('activate_imunify');
+    $serviceExtra = run_event('parse_service_extra', $serviceInfo[$settings['PREFIX'].'_extra'], self::$module);
+    activate_imunify($serviceInfo[$settings['PREFIX'].'_ip'], $pass, $GLOBALS['tf']->accounts->cross_reference($serviceInfo[$settings['PREFIX'].'_custid']), self::$module.$serviceInfo[$settings['PREFIX'].'_id']);
+}
+```
+
+## Common Issues
+
+- **`Call to undefined function activate_*`**: You skipped `function_requirements('activate_*')` before calling it. Add the call immediately before the activation function call.
+- **`Undefined index: PREFIX_id`**: `get_module_settings()` was not called, or `self::$module` is wrong. Verify `public static $module = 'vps';` matches the registered module name.
+- **`require_once` path resolves to wrong location**: `__DIR__` is relative to `src/`. The path `__DIR__.'/../../../../include/licenses/license.functions.inc.php'` assumes four directory levels up to the MyAdmin root. If the plugin is not installed under `vendor/detain/myadmin-*/src/`, adjust the `../` count.
+- **Admin email not sent after disable**: Ensure `(new \MyAdmin\Mail())` is instantiated with `new` and the template path is relative to `include/templates/email/`. Missing template file will silently fail — confirm the `.tpl` exists.
+- **`vps_get_password` returns empty**: The VPS record may lack a stored password. Log `$pass` with `myadmin_log()` immediately after the call to confirm before passing to the activation function.

.claude/skills/find-skills/SKILL.md

@@ -0,0 +1,54 @@
+---
+name: find-skills
+description: Discovers and installs community skills from the public registry. Use when the user mentions a technology, framework, or task that could benefit from specialized skills not yet installed, asks 'how do I do X', 'find a skill for X', or starts work in a new technology area. Proactively suggest when the user's task involves tools or frameworks without existing skills.
+---
+
+# Find Skills
+
+Search the public skill registry for community-contributed skills
+relevant to the user's current task and install them into this project.
+
+## Instructions
+
+1. Identify the key technologies, frameworks, or task types from the
+   user's request that might have community skills available
+2. Ask the user: "Would you like me to search for community skills
+   for [identified technologies]?"
+3. If the user agrees, run:
+   ```bash
+   caliber skills --query "<relevant terms>"
+   ```
+   This outputs the top 5 matching skills with scores and descriptions.
+4. Present the results to the user and ask which ones to install
+5. Install the selected skills:
+   ```bash
+   caliber skills --install <slug1>,<slug2>
+   ```
+6. Read the installed SKILL.md files to load them into your current
+   context so you can use them immediately in this session
+7. Summarize what was installed and continue with the user's task
+
+## Examples
+
+User: "let's build a web app using React"
+-> "I notice you want to work with React. Would you like me to search
+   for community skills that could help with React development?"
+-> If yes: run `caliber skills --query "react frontend"`
+-> Show the user the results, ask which to install
+-> Run `caliber skills --install <selected-slugs>`
+-> Read the installed files and continue
+
+User: "help me set up Docker for this project"
+-> "Would you like me to search for Docker-related skills?"
+-> If yes: run `caliber skills --query "docker deployment"`
+
+User: "I need to write tests for this Python ML pipeline"
+-> "Would you like me to find skills for Python ML testing?"
+-> If yes: run `caliber skills --query "python machine-learning testing"`
+
+## When NOT to trigger
+
+- The user is working within an already well-configured area
+- You already suggested skills for this technology in this session
+- The user is in the middle of urgent debugging or time-sensitive work
+- The technology is too generic (e.g. just "code" or "programming")

.claude/skills/phpunit-source-test/SKILL.md

@@ -0,0 +1,109 @@
+---
+name: phpunit-source-test
+description: Writes PHPUnit 9 source-reading tests that verify PHP file structure without executing code. Use when user says 'write tests', 'add test for', or adds/modifies files in src/. Tests use file_get_contents() + assertStringContainsString/assertMatchesRegularExpression for procedural files, and ReflectionClass for class files. Do NOT use for integration tests, mock-based tests, or tests that exercise runtime behavior.
+---
+# PHPUnit Source-Reading Tests
+
+## Critical
+
+- Never instantiate classes or call functions from `src/` directly — they depend on heavy MyAdmin globals (`function_requirements`, DB, constants). Use `file_get_contents()` or `ReflectionClass` only.
+- Test namespace must be `Detain\MyAdminVpsDirectadmin\Tests` — matches `tests/` directory in `composer.json`.
+- Bootstrap is configured via `phpunit.xml.dist` — do not add require statements.
+- Run tests with: `vendor/bin/phpunit tests/ -v`
+
+## Instructions
+
+1. **Create the test file** in `tests/` named after the class or file under test (e.g., `tests/PluginTest.php`). Use `tests/VpsAddDirectadminTest.php` as the template for procedural files; use `tests/PluginTest.php` for class files.
+
+2. **Set up the class** with the correct namespace, import, and class declaration:
+   ```php
+   <?php
+   namespace Detain\MyAdminVpsDirectadmin\Tests;
+   use PHPUnit\Framework\TestCase;
+   // For class files also add:
+   use ReflectionClass;
+   use ReflectionMethod;
+   
+   class MyFileTest extends TestCase
+   ```
+   Verify the namespace matches `tests/` autoload entry in `composer.json` before proceeding.
+
+3. **For procedural `src/*.php` files** — add a `setUp()` that loads the source into `$this->source`:
+   ```php
+   private string $filePath;
+   private string $source;
+   
+   protected function setUp(): void
+   {
+       $this->filePath = dirname(__DIR__) . '/src/my_function.php';
+       $this->assertFileExists($this->filePath);
+       $this->source = file_get_contents($this->filePath);
+   }
+   ```
+   Then write test methods using:
+   - `$this->assertStringStartsWith('<?php', $this->source)` — PHP open tag
+   - `$this->assertMatchesRegularExpression('/function\s+my_function\s*\(/', $this->source)` — function declaration
+   - `$this->assertStringContainsString("function_requirements('class.Foo')", $this->source)` — required calls
+   - `$this->assertStringNotContainsString('namespace ', $trimmed)` — no namespace in procedural files (skip comment lines with `str_starts_with($trimmed, '*')` etc.)
+
+4. **For class files** — use `ReflectionClass` in `setUp()`:
+   ```php
+   private ReflectionClass $ref;
+   
+   protected function setUp(): void
+   {
+       $this->ref = new ReflectionClass(Plugin::class);
+   }
+   ```
+   Cover: `isInstantiable()`, `isStatic()`/`isPublic()` on methods, `getParameters()` count and names, static property values (`Plugin::$module`), `getHooks()` array keys and callable shapes `[ClassName::class, 'methodName']`.
+   For source-level checks on class files: `file_get_contents($this->ref->getFileName())`.
+
+5. **Group tests with comment separators** matching existing style:
+   ```php
+   // ------------------------------------------------------------------
+   //  Function declaration
+   // ------------------------------------------------------------------
+   ```
+   Standard groups: File existence and structure · Function/class declaration · Internal calls · Docblock and metadata · File-level characteristics.
+
+6. **Every test method** must be `public function testXxx(): void` with a one-line docblock describing what it asserts. Verify all assertions pass by running: `vendor/bin/phpunit tests/PluginTest.php -v`
+
+## Examples
+
+**User says:** "Write tests for the new `src/vps_add_cpanel.php` procedural file."
+
+**Actions taken:**
+1. Read `src/vps_add_cpanel.php` to identify: function name, `function_requirements()` calls, method chains, constants used.
+2. Create `tests/VpsAddCpanelTest.php` with `setUp()` loading the file into `$this->source`.
+3. Add test methods: `testFileExists`, `testFileStartsWithPhpTag`, `testFileHasNoNamespace`, `testDeclaresFunctionSignature`, `testRequiresAddServiceAddon`, `testCallsLoadMethod`, `testCallsProcessMethod`, `testFileHasDocblock`.
+
+**Result:**
+```php
+protected function setUp(): void
+{
+    $this->filePath = dirname(__DIR__) . '/src/vps_add_cpanel.php';
+    $this->assertFileExists($this->filePath);
+    $this->source = file_get_contents($this->filePath);
+}
+
+public function testDeclaresVpsAddCpanelFunction(): void
+{
+    $this->assertMatchesRegularExpression(
+        '/function\s+vps_add_cpanel\s*\(\s*\)/',
+        $this->source
+    );
+}
+
+public function testRequiresAddServiceAddon(): void
+{
+    $this->assertStringContainsString("function_requirements('class.AddServiceAddon')", $this->source);
+}
+```
+
+## Common Issues
+
+- **`Class 'Detain\MyAdminVpsDirectadmin\Plugin' not found`**: `composer install` has not been run, or autoload is stale. Run `composer dump-autoload`.
+- **`assertMatchesRegularExpression` not found**: You are using PHPUnit < 9. This project uses PHPUnit 9 — check `composer.json` requires `phpunit/phpunit ^9`.
+- **`file_get_contents(): Failed to open stream`**: Path built with `dirname(__DIR__)` is wrong. Confirm test file lives in `tests/` (one level below project root) so `dirname(__DIR__)` resolves to the repo root.
+- **Test passes locally but CI fails with undefined constant `VPS_DA_COST`**: You called the actual function. Source-reading tests must only use `file_get_contents()` / `assertStringContainsString` — never `include` or `require` the `src/` file under test.
+- **`str_starts_with` undefined**: Requires PHP 8.0+. If running PHP 7.4, replace with `strncmp($trimmed, '*', 1) === 0`.