opencode-forgekeeper 1.0.0 → 2.0.0

package/README.md

@@ -296,8 +296,9 @@ title prefix. It doesn't poll or send review notifications, and the CLI shows no
 
 ## RPC
 
-The server registers an [RPC](https://opencode.ai/v2/docs/build/plugins/rpc) that the CLI
-uses, defined in `src/rpc.ts`:
+The server registers an [RPC](https://opencode.ai/v2/docs/build/plugins/rpc) with the ID
+`opencode-forgekeeper`, defined in `src/rpc.ts`. The CLI uses it, and other plugins can call
+`target` and listen for `targetChanged`:
 
 - `target({ sessionID })` returns `{ url, issueUrl?, targets }` for explicit targets, and
   `{}` in branch mode. `targets` lists every target as `{ url, issueUrl? }`. `url` and
@@ -318,18 +319,25 @@ plugin instance watches its own sessions.
 
 ## Migrating from opencode-forge-session-title
 
-Replace `opencode-forge-session-title` with `opencode-forgekeeper` in `opencode.json`. Never
-load both: the server keeps `opencode-forge-session-title`'s plugin and RPC IDs, so the two
-would collide.
-
-| Entry point | Plugin ID                      | Stored state                                                                             |
-| ----------- | ------------------------------ | ---------------------------------------------------------------------------------------- |
-| Server      | `opencode-forge-session-title` | Session targets, owned title prefixes, human-review baselines, and pending notifications |
-| CLI         | `pedropombeiro.forgekeeper`    | None                                                                                     |
-
-Because the IDs match, stored targets carry over, and callers of the `target` RPC keep working.
-The server reads a session's single stored `target` as a list of one, and writes `targets` from
-the next change on.
+Replace `opencode-forge-session-title` with `opencode-forgekeeper` in `opencode.json`. Upgrading
+from `opencode-forgekeeper` 1.x works the same way, because 1.x used the same IDs as
+`opencode-forge-session-title`.
+
+| Entry point | Plugin ID                   | Stored state                                                                             |
+| ----------- | --------------------------- | ---------------------------------------------------------------------------------------- |
+| Server      | `opencode-forgekeeper`      | Session targets, owned title prefixes, human-review baselines, and pending notifications |
+| CLI         | `pedropombeiro.forgekeeper` | None                                                                                     |
+
+OpenCode scopes plugin storage by plugin ID, so nothing stored under the former ID,
+`opencode-forge-session-title`, carries over:
+
+- Sessions start in branch-based naming. Ask the agent to set their targets again.
+- A title that kept a single-reference prefix from before, such as `[!456] Review`, can get a
+  second prefix, such as `[#12, !45] [!456] Review`. Remove the older one by hand. Older prefixes
+  with two references, such as `[#12, !45]`, are replaced automatically.
+- The first look at each PR/MR records a new review baseline, so comments posted before then
+  aren't announced.
+- Callers of the RPC need the new ID, `opencode-forgekeeper`.
 
 With default options, the new footer and review notifications start right away, and
 notifications resume the session. Set `reviewStatus` to `false` to keep only titles, or

package/dist/index.js

@@ -917,7 +917,7 @@ var status = {
   required: ["enabled", "snapshot"]
 };
 var ForgeSessionRpc = {
-  id: "opencode-forge-session-title",
+  id: "opencode-forgekeeper",
   methods: {
     target: {
       input: {
@@ -1805,7 +1805,6 @@ var FEEDBACK_PREFIX = "status/feedback/";
 var pendingPrefix = (sessionID) => `${PENDING_PREFIX}${encodeURIComponent(sessionID)}/`;
 var pendingKey = (sessionID, url) => `${pendingPrefix(sessionID)}${encodeURIComponent(url)}`;
 var feedbackRecordKey = (key2) => `${FEEDBACK_PREFIX}${encodeURIComponent(key2)}`;
-var LEGACY_KEYS = ["status/humanReviewFeedback", "status/automatedReviewPending"];
 function userTitle(title, prefix) {
   if (!title || !prefix)
     return title;
@@ -1971,8 +1970,6 @@ async function setupStatus(ctx, settings, deps, host) {
   });
   invalidate = (sessionID) => service.invalidate(sessionID);
   write(async () => {
-    for (const key2 of LEGACY_KEYS)
-      await ctx.storage.remove(key2);
     const at = (deps.now ?? Date.now)();
     for (const { key: key2, value } of await scanAll(FEEDBACK_PREFIX)) {
       if (isExpired(value, at))
@@ -2095,7 +2092,7 @@ async function setupServer(ctx, lookups, status2) {
   async function stateFor(sessionID) {
     const started = revision(sessionID);
     const stored = await ctx.storage.get(`sessions/${sessionID}`) ?? {};
-    const targets = Array.isArray(stored.targets) ? stored.targets : stored.target ? [stored.target] : [];
+    const targets = Array.isArray(stored.targets) ? stored.targets : [];
     if (revision(sessionID) === started)
       known.set(sessionID, targets);
     return { targets, ...stored.prefix ? { prefix: stored.prefix } : {} };
@@ -2299,7 +2296,7 @@ async function setupServer(ctx, lookups, status2) {
 
 // packages/forgekeeper/src/index.ts
 var src_default = {
-  id: "opencode-forge-session-title",
+  id: "opencode-forgekeeper",
   async setup(ctx) {
     const hosts = hostsFrom(ctx.options ?? {});
     const stopped = new AbortController;

package/dist/tui.js

@@ -883,7 +883,7 @@ var status = {
   required: ["enabled", "snapshot"]
 };
 var ForgeSessionRpc = {
-  id: "opencode-forge-session-title",
+  id: "opencode-forgekeeper",
   methods: {
     target: {
       input: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-forgekeeper",
-  "version": "1.0.0",
+  "version": "2.0.0",
   "description": "OpenCode plugin that tracks each session's GitHub and GitLab issues and PRs/MRs: title prefixes, a PR/MR status footer, and review notifications",
   "author": {
     "name": "Pedro Pombeiro",