feat(snapshot): gcSnapshots + ocp update --rollback --gc + auto-GC; v3.16.0

Adds snapshot garbage collection with retention policy: keep last 5, keep snapshots within 30 days, always keep the most recent. Configurable via keepCount / keepDays opts. Wire-up: - ocp update --rollback --gc (manual trigger; --dry-run supported) - runFullUpgrade auto-GC after successful upgrade (best-effort, swallows errors) 4 unit tests: keepCount enforcement, keepDays override, never-delete- most-recent safety, dry-run mode. Bumps to v3.16.0 (bundles PR #93 --check oauth + this GC feature). No cli.js citation needed: this is OCP-internal snapshot lifecycle with no corresponding cli.js operation. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-07-22 13:35:08 +00:00 · 2026-05-11 07:29:53 +10:00
7 changed files with 11 additions and 81 deletions
@@ -1,65 +1,6 @@
 # Changelog

-## v3.16.3 — 2026-05-13
-
-### Fixes — completes v3.16.2 port-drift revert
-
-v3.16.2 reverted the plugin / `openclaw.plugin.json` / README / Mac mini
-plist back to `3456` (the historical source default since `593d0dc`), but
-missed three places in `scripts/` that still defaulted to `3478`. Those
-three lines were the residual cascade source: every time `ocp doctor` or
-`ocp upgrade` ran without `CLAUDE_PROXY_PORT` in the env, they probed
-`3478`, reported "OCP not responding" against a healthy 3456 instance,
-and (in the case of OpenClaw sync follow-ups on the maintainer's host)
-re-introduced 3478 into downstream config.
-
-Changes:
-
- `scripts/upgrade.mjs:137` — default port `3478` → `3456`.
- `scripts/doctor.mjs:84` — default port `3478` → `3456`.
- `scripts/doctor.mjs:205` — default port `3478` → `3456`.
-
-No behavior change for users who set `CLAUDE_PROXY_PORT` explicitly; env
-still takes precedence. The fix only affects the unset-env fallback,
-which now matches `server.mjs:126` and the rest of the codebase.
-
-Test plan: existing `test-features.mjs` cases that pin
-`CLAUDE_PROXY_PORT=3478` continue to pass — they use the env path, not
-the default.
-
-## v3.16.2 — 2026-05-12
-
-### Fixes — corrects v3.16.1
-
-The v3.16.1 fix was directionally correct (plugin now reads env first, falls back to a hardcoded default) but **the narrative and the hardcoded default were both wrong**.
-
-What v3.16.1 said: "OCP server moved to 3478 default in v3.14+; plugin lagged at 3456."
-What is actually true:
- **OCP server source default has been `3456` since `593d0dc` (initial release) and has never changed.** Every line in `server.mjs`, `setup.mjs`, and the `ocp` CLI still uses `3456` as the documented and code-level default.
- The single OCP installation observed on `3478` is the maintainer's Mac mini, whose plist was rewritten with `--port 3478` during a PR #71 dogfood smoke-test accident on 2026-05-08 (see `~/.cc-rules/memory/learnings/subagent_setup_mjs_prod_host_collision.md`). The plist drift was never reconciled back to source default, and v3.16.1 incorrectly canonised the post-accident value as if it had been a release decision.
-
-This release:
- Restores the plugin fallback to `http://127.0.0.1:3456` to match server source default.
- Updates `openclaw.plugin.json` `configSchema.proxyUrl.default` back to `3456`.
- Restores README §"Environment Variables" `CLAUDE_PROXY_PORT` default to `3456`.
- Plugin reads `OCP_PROXY_URL` env (full URL) first, then `CLAUDE_PROXY_PORT` env (port only), then falls back to `3456`. Hosts whose OCP plist injects a non-default port must also inject the same `CLAUDE_PROXY_PORT` into the OpenClaw plist for the plugin to follow.
- Maintainer's Mac mini plist was reverted from `3478` to `3456` as part of this release deploy (no source change reflects this; it was a one-host correction).
-
-### Governance
-
- No `cli.js` citation needed (no `server.mjs` change). ALIGNMENT.md Rule 2 not engaged.
-
-## v3.16.1 — 2026-05-12 (superseded — narrative incorrect; see v3.16.2 erratum)
-
-### Fixes (as shipped — note erratum above)
-
- **OCP plugin port lag** — `ocp-plugin/index.js` hard-coded `http://127.0.0.1:3456`. ~~While OCP server moved to 3478 in v3.14+,~~ **(corrected v3.16.2: no such move ever happened.)** The Mac mini's plist was on `3478` only as residue from a dogfood accident. Result: `/ocp` slash commands from the home Telegram bot returned "OCP error: fetch failed". v3.16.1 changed the plugin default to `3478` (wrong direction; v3.16.2 reverts to `3456`).
-
-### Governance
-
- No `cli.js` citation needed (no `server.mjs` change). ALIGNMENT.md Rule 2 not engaged.
-
-## v3.16.0 — 2026-05-10
+## v3.16.0 — 2026-XX-XX (release date filled at tag time)

 ### Features

@@ -855,8 +855,7 @@ Future `ocp update` invocations sync automatically.

 | Variable | Default | Description |
 |----------|---------|-------------|
-| `CLAUDE_PROXY_PORT` | `3456` | Listen port (server-side). Also consumed by the OpenClaw `ocp-plugin` to dial the local proxy. |
-| `OCP_PROXY_URL` | *(unset)* | Plugin-side full URL override (e.g. `http://10.0.0.5:3456`). Wins over `CLAUDE_PROXY_PORT` when both are set. Read by `ocp-plugin/index.js` only — server ignores it. |
+| `CLAUDE_PROXY_PORT` | `3456` | Listen port |
 | `CLAUDE_BIND` | `127.0.0.1` | Bind address (`0.0.0.0` for LAN access) |
 | `CLAUDE_AUTH_MODE` | `none` | Auth mode: `none`, `shared`, or `multi` |
 | `OCP_ADMIN_KEY` | *(unset)* | Admin key for key management (multi mode) |
@@ -1,19 +1,9 @@
 /**
 * OCP Plugin — registers /ocp as a native slash command in OpenClaw gateway.
- * Calls the local claude-proxy and formats the response.
- *
- * Port resolution (in priority order):
- *   1. OCP_PROXY_URL env (full URL, e.g. http://10.0.0.5:3456)
- *   2. CLAUDE_PROXY_PORT env (port only; localhost assumed)
- *   3. Fallback: http://127.0.0.1:3456 (OCP server source default since v1.0)
- *
- * If a particular host's OCP plist injects a non-default CLAUDE_PROXY_PORT,
- * the OpenClaw launchd plist for that host must also inject the same
- * CLAUDE_PROXY_PORT into the plugin's env, or the plugin will fall back to
- * 3456 and miss the server.
+ * Calls the local claude-proxy at http://127.0.0.1:3456 and formats the response.
 */
-const PROXY = process.env.OCP_PROXY_URL
-  || (process.env.CLAUDE_PROXY_PORT ? `http://127.0.0.1:${process.env.CLAUDE_PROXY_PORT}` : "http://127.0.0.1:3456");
+
+const PROXY = "http://127.0.0.1:3456";

 // Wrap output in monospace code block for Telegram/Discord alignment
 function mono(text) { return "```\n" + text + "\n```"; }
@@ -2,7 +2,7 @@
  "id": "ocp",
  "name": "OCP Commands",
  "description": "Slash commands for the OpenClaw Proxy — /ocp usage, /ocp settings, /ocp health, etc.",
-  "version": "3.16.2",
+  "version": "3.12.0",
  "configSchema": {
    "type": "object",
    "additionalProperties": false,
@@ -10,7 +10,7 @@
      "proxyUrl": {
        "type": "string",
        "default": "http://127.0.0.1:3456",
-        "description": "URL of the Claude proxy. Overridable via OCP_PROXY_URL or CLAUDE_PROXY_PORT env."
+        "description": "URL of the Claude proxy"
      }
    }
  }
@@ -1,6 +1,6 @@
 {
  "name": "open-claude-proxy",
-  "version": "3.16.3",
+  "version": "3.16.0",
  "description": "OCP (Open Claude Proxy) — use your Claude Pro/Max subscription as an OpenAI-compatible API for any IDE. Works with Cline, OpenCode, Aider, Continue.dev, OpenClaw, and more.",
  "type": "module",
  "bin": {
@@ -81,7 +81,7 @@ export async function runDoctor(opts = {}) {
      health = opts.mockHealth;
    } else {
      try {
-        const port = process.env.CLAUDE_PROXY_PORT || "3456";
+        const port = process.env.CLAUDE_PROXY_PORT || "3478";
        const out = execSync(`curl -sf --max-time 3 http://127.0.0.1:${port}/health`, { stdio: ["pipe", "pipe", "pipe"] }).toString();
        health = { status: 200, body: JSON.parse(out) };
      } catch (e) {
@@ -202,7 +202,7 @@ function runOauthOnly(opts, checks, push) {
    health = opts.mockHealth;
  } else {
    try {
-      const port = process.env.CLAUDE_PROXY_PORT || "3456";
+      const port = process.env.CLAUDE_PROXY_PORT || "3478";
      const out = execSync(`curl -sf --max-time 3 http://127.0.0.1:${port}/health`, { stdio: ["pipe", "pipe", "pipe"] }).toString();
      health = { status: 200, body: JSON.parse(out) };
    } catch (e) {
@@ -134,7 +134,7 @@ async function runFullUpgrade({ doctor, opts }) {

    // phase 6: post-flight (10s budget; skipped under mockExec)
    if (!opts.mockExec) {
-      const port = process.env.CLAUDE_PROXY_PORT || "3456";
+      const port = process.env.CLAUDE_PROXY_PORT || "3478";
      let ok = false;
      for (let i = 0; i < 10; i++) {
        try {