Compare commits

..

No commits in common. "c2a082b7805b522e920e378df06d06c04f52ba8c" and "7e9d372d96897944b4c9d54f48e260c1aa75ec96" have entirely different histories.

3 changed files with 3 additions and 5 deletions

View File

@ -1,6 +1,6 @@
{ {
"name": "@because/sandlot", "name": "@because/sandlot",
"version": "0.0.13", "version": "0.0.12",
"description": "Sandboxed, branch-based development with Claude", "description": "Sandboxed, branch-based development with Claude",
"type": "module", "type": "module",
"bin": { "bin": {

View File

@ -148,7 +148,6 @@ program
program program
.command("review") .command("review")
.argument("<branch>", "branch name") .argument("<branch>", "branch name")
.argument("[prompt]", "additional instructions to append to the review prompt")
.option("-p, --print", "print the review to stdout instead of launching interactive mode") .option("-p, --print", "print the review to stdout instead of launching interactive mode")
.description("Launch an interactive grumpy code review for a branch") .description("Launch an interactive grumpy code review for a branch")
.action(reviewAction) .action(reviewAction)

View File

@ -2,14 +2,13 @@ import * as vm from "../vm.ts"
import { spinner } from "../spinner.ts" import { spinner } from "../spinner.ts"
import { requireSession, saveChanges } from "./helpers.ts" import { requireSession, saveChanges } from "./helpers.ts"
export async function action(branch: string, extra: string | undefined, opts: { print?: boolean }) { export async function action(branch: string, opts: { print?: boolean }) {
const { session } = await requireSession(branch) const { session } = await requireSession(branch)
const spin = spinner("Starting container", branch) const spin = spinner("Starting container", branch)
await vm.ensure((msg) => { spin.text = msg }) await vm.ensure((msg) => { spin.text = msg })
let prompt = "You're a grumpy old senior software engineer. Take a look at the diff between this branch and main, then let me know your thoughts. My co-worker made these changes." const prompt = "You're a grumpy old senior software engineer. Take a look at the diff between this branch and main, then let me know your thoughts. My co-worker made these changes."
if (extra) prompt += "\n\n" + extra
if (opts.print) { if (opts.print) {
spin.text = "Running review…" spin.text = "Running review…"