npm - usebeeline - Versions diffs - 0.0.98 → 0.0.99 - Mend

usebeeline 0.0.98 → 0.0.99

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (2) hide show

package/dist/usebeeline.mjs +16 -17
package/package.json +1 -1

package/dist/usebeeline.mjs CHANGED Viewed

@@ -18373,8 +18373,7 @@ Follow these steps in order. Do not skip or reorder them.
 ## 8. Gate and verdict
-- The merge gate is open only when \`pr_checks_status\` reports checks=passed, held=false, approvalPending=false.
-- Green \`gh pr checks\` alone never opens the gate.
+- Review the exact green head named in your reviewer instruction. If the head moved, do not approve it.
 - Always use this exact verdict shape:
 \`objective quoted:\`
@@ -18388,11 +18387,9 @@ Follow these steps in order. Do not skip or reorder them.
 Then take exactly one action:
-- FAIL: reply \`@author\` with the confirmed findings; do not merge.
-- PASS with pending checks: reply \`approved pending checks <reviewed sha>\` and stop.
-- PASS with unknown checks: report the \`pr_checks_status\` reason to the Room and stop; do not retry.
-- PASS with the gate open and your yolo on: run \`gh pr merge --squash --match-head-commit <reviewed sha> N\`.
-- PASS with the gate open and your yolo off: reply \`approved <reviewed sha>\` and stop.
+- FAIL: reply \`@author\` with the confirmed findings to fix.
+- PASS: call \`approve_merge\` with the reviewed head SHA, then reply \`@author approved <reviewed sha>, merge\`.
+- Never merge the pull request yourself.
 `;
 }
@@ -20374,6 +20371,7 @@ function beelineAgentMcpServer(config, api, context) {
       { name: "BEELINE_DAEMON_ROOM_ID", value: context.roomId },
       { name: "BEELINE_DAEMON_WORKSPACE_ID", value: context.workspaceId },
       ...context.cornerId ? [{ name: "BEELINE_DAEMON_CORNER_ID", value: context.cornerId }] : [],
+      ...context.reviewer ? [{ name: "BEELINE_CORNER_REVIEWER", value: "1" }] : [],
       ...context.attachRoot ? [{ name: "BEELINE_ATTACH_ROOT", value: context.attachRoot }] : [],
       ...context.attachScratchRoot ? [{ name: "BEELINE_ATTACH_SCRATCH_ROOT", value: context.attachScratchRoot }] : [],
       ...context.grantRunner ? [
@@ -22091,7 +22089,7 @@ var TOOL_OUTPUT_MAX_BYTES = 3200;
 var TOOL_PATH_LIMIT = 12;
 function cornerMergeInstruction(yoloMode, reviewerHandle) {
   if (reviewerHandle)
-    return `Commit, push, open the pull request, reply with the PR URL and then @${reviewerHandle} please review; never merge this PR yourself. If you asked any other agent to review in this corner, do not merge until they answer.`;
+    return `Commit, push, open the PR, and reply with the URL; do not merge until @${reviewerHandle} tags you with approval, then merge with gh pr merge --squash --match-head-commit <sha>.`;
   return yoloMode ? "Yolo is on: when the gate passes, merge this pull request with gh." : "Yolo is off: never merge; wait for explicit human approval in the app.";
 }
 function cornerReviewerInstruction(input) {
@@ -22099,7 +22097,8 @@ function cornerReviewerInstruction(input) {
     return void 0;
   const author = input.authorHandle?.replace(/^@/, "") || "author";
   const number = input.pullRequestNumber ?? "N";
-  return `Review PR #${number} with the beeline-review skill. If it fails, reply @${author} with the findings. If it passes and the gate (pr_checks_status: checks=passed, held=false, approvalPending=false) is open and YOUR yolo is on, merge with gh pr merge --squash --match-head-commit <sha you reviewed>; if your yolo is off, reply approved <sha> and stop; if checks are pending, reply approved pending checks <sha> and stop; if checks are unknown, report the tool's reason to the Room and stop instead of retrying.`;
+  const headSha = input.headSha ?? "<head sha>";
+  return `Checks are green on PR #${number} at ${headSha}. Review it now with the beeline-review skill against that exact head. FAIL: reply \`@${author}\` with the confirmed findings to fix. PASS: call the approve_merge tool for ${headSha}, then reply \`@${author} approved ${headSha}, merge\`. Never merge yourself. Never say you are holding or waiting for checks.`;
 }
 var CORNER_AUTHOR_CONTRACT = `The objective text is the user's ask. Keep it verbatim in your head and do not reinterpret it.
 Before any code, write its end-user story in one sentence: "a person who does X sees Y".
@@ -22114,7 +22113,7 @@ Under ## Demonstrated, give the command or steps that produced Y and what was ob
 A pull request without both sections is not deliverable and the Room's reviewer will fail it.
 Change only what the objective asks. No unrequested features, flags, compatibility shims, or refactors.`;
 var CORNER_DELIVERY_NUDGE = "Before ending this turn, inspect the repository state and finish delivering the work: commit and push the intended changes and open the pull request if one does not exist. Decide yourself whether any remaining dirty work belongs to the objective; do not discard it merely to make the worktree clean. The pull request body must carry ## Reproduced and ## Demonstrated; if they are missing, add them before ending the turn.";
-var CORNER_YOLO_MERGE_NUDGE = 'Yolo is on. Check the server merge gate with pr_checks_status now and, if checks="passed", held=false, and approvalPending=false, merge this pull request with gh. If checks="unknown", report the tool reason to the Room and stop instead of retrying. Otherwise stop without merging.';
+var CORNER_YOLO_MERGE_NUDGE = 'Yolo is on. Check the server merge gate with pr_checks_status now and, if checks="passed", held=false, and approvalPending=false, merge this pull request with gh. If checks="unknown", reply in this corner with the tool reason and stop instead of retrying. Otherwise stop without merging.';
 function isCornerChecksTurn(trigger, restates) {
   return Boolean(restates) || /\b(?:passed|failed) a check\b/i.test(trigger);
 }
@@ -22450,8 +22449,7 @@ var MonolithCornerTurnLoop = class {
       reviewerHandle: configuration.reviewerHandle,
       agentHandle: self?.handle,
       authorHandle: opener?.handle,
-      openedByAgent: !this.options.openedBy || this.options.openedBy === this.agent.publicKey,
-      yoloMode: configuration.yoloMode
+      openedByAgent: !this.options.openedBy || this.options.openedBy === this.agent.publicKey
     };
     let reviewerInstruction = cornerReviewerInstruction(reviewerInput);
     if (reviewerInstruction) {
@@ -22460,7 +22458,8 @@ var MonolithCornerTurnLoop = class {
       });
       reviewerInstruction = cornerReviewerInstruction({
         ...reviewerInput,
-        pullRequestNumber: restore.lifecycle?.pr?.number
+        pullRequestNumber: restore.lifecycle?.pr?.number,
+        headSha: restore.lifecycle?.pr?.headSha
       });
     }
     this.cornerTurnEndNudge = reviewerInstruction ?? cornerMergeInstruction(configuration.yoloMode, configuration.reviewerHandle);
@@ -22579,6 +22578,7 @@ var MonolithCornerTurnLoop = class {
         roomId: this.options.parentRoomId,
         workspaceId: this.options.workspaceId,
         cornerId: this.options.cornerId,
+        reviewer: Boolean(reviewerInstruction),
         attachRoot: this.options.worktreePath,
         // The whole per-session overlay, not an enumerated subset: see
         // `monolith-room-turn.ts`'s matching comment.
@@ -22596,8 +22596,7 @@ var MonolithCornerTurnLoop = class {
     const identityInstructions = `Your Beeline identity is ${self?.name ?? this.agent.name}.`;
     const personaInstructions = [
       ...persona?.instructions ? [`Human-authored Workspace persona: ${persona.name}. ${persona.instructions}`] : [],
-      SOUL_HOUSE_RULE,
-      "Report milestones, blockers, and questions to the Room with report_to_room; do not narrate."
+      SOUL_HOUSE_RULE
     ].join("\n");
     this.turnIdentityInstructions = harnessHonorsSessionSystemPrompt(command) ? "" : [identityInstructions, personaInstructions].filter(Boolean).join("\n\n");
     const opened = await this.client.sessionNew({
@@ -22611,12 +22610,12 @@ var MonolithCornerTurnLoop = class {
           `You are in an isolated git worktree on ${repository.featureBranch}, targeting ${repository.targetBranch}.`,
           `Commit and push only ${repository.featureBranch}; never force-push or write to ${repository.targetBranch}. Before pushing, rebase on origin/${repository.featureBranch}; resolve conflicts autonomously, realigning to that remote branch and redoing the objective if needed, then rerun affected tests. Open the pull request with gh.`,
           ...reviewerInstruction ? [reviewerInstruction] : [
-            configuration.reviewerHandle ? `Once the pull request exists, reply with its full URL, then @${configuration.reviewerHandle} please review, and end the turn; do not check or wait for CI.` : 'Once the pull request exists, reply only with its full URL and end the turn; do not check or wait for CI. On a later checks turn, call pr_checks_status. Merge only when checks="passed", held=false, and approvalPending=false; if checks="unknown", report the tool reason to the Room and stop instead of retrying. Only a later explicit human resume clears a hold.',
+            configuration.reviewerHandle ? `Once the pull request exists, reply with its full URL and end the turn; do not tag the reviewer, check, or wait for CI.` : 'Once the pull request exists, reply only with its full URL and end the turn; do not check or wait for CI. On a later checks turn, call pr_checks_status. Merge only when checks="passed", held=false, and approvalPending=false; if checks="unknown", reply in this corner with the tool reason and stop instead of retrying. Only a later explicit human resume clears a hold.',
             CORNER_AUTHOR_CONTRACT,
             cornerMergeInstruction(configuration.yoloMode, configuration.reviewerHandle)
           ],
           "Do not tag the user when a corner turn finishes: the server posts the merge summary card and its push already cover completion. Tag a human only mid-turn, and only when you need a decision or input.",
-          "Never restate server check or merge notes. On a checks turn, say nothing unless you merge or push a fix, then use one short line. When approval is pending, wait for the server close request. Never merge another pull request."
+          "Never restate server check or merge notes. On a checks turn, say nothing unless you merge or push a fix, then use one short line. Never merge while approvalPending is true. When approval is pending, wait for the reviewer to tag you. Never merge another pull request."
         ] : [
           "This is a chat-only corner with no repository or GitHub workflow.",
           "Work in this corner's writable workspace. Use write_scratch_file or ordinary tools to create files, then attach_file to send them back to the corner.",

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "usebeeline",
-  "version": "0.0.98",
+  "version": "0.0.99",
   "description": "Connect an AI coding agent to Beeline with one command.",
   "homepage": "https://usebeeline.app",
   "bugs": {