super-review 0.0.1 → 0.0.3
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.
- package/dist/bin.mjs +41 -8
- package/package.json +4 -2
package/dist/bin.mjs
CHANGED
|
@@ -1,13 +1,33 @@
|
|
|
1
1
|
#!/usr/bin/env node
|
|
2
|
-
import{Command as e,program as t}from"commander";import n from"node:path";import{simpleGit as r}from"simple-git";import{createHash as i,randomUUID as a}from"node:crypto";import{promises as
|
|
3
|
-
`).filter(Boolean)){let t=e.split(` `),n=t[0],i=`modified`,
|
|
4
|
-
`).filter(Boolean)){let t=e.split(` `),n=t[0],r=`modified`,i,o=t[1];n.startsWith(`A`)?r=`added`:n.startsWith(`D`)?r=`deleted`:n.startsWith(`M`)?r=`modified`:n.startsWith(`R`)?(r=`renamed`,i=t[1],o=t[2]):n.startsWith(`C`)?(r=`copied`,i=t[1],o=t[2]):n.startsWith(`T`)&&(r=`type-change`);let c
|
|
5
|
-
`).filter(Boolean)){let
|
|
2
|
+
import{Command as e,program as t}from"commander";import n from"node:path";import{simpleGit as r}from"simple-git";import{createHash as i,randomUUID as a}from"node:crypto";import{mkdirSync as o,promises as s}from"node:fs";import{execFile as c}from"node:child_process";import{promisify as l}from"node:util";import{homedir as u}from"node:os";import{createClient as d}from"@libsql/client";import{drizzle as f}from"drizzle-orm/libsql";import{and as p,desc as m,eq as h}from"drizzle-orm";import{integer as g,sqliteTable as _,text as v}from"drizzle-orm/sqlite-core";var y=`0.0.3`,b=`Document an agent session for review with the super-review desktop app`;const ee={png:`image/png`,jpg:`image/jpeg`,jpeg:`image/jpeg`,gif:`image/gif`,webp:`image/webp`,avif:`image/avif`,bmp:`image/bmp`,ico:`image/x-icon`,svg:`image/svg+xml`};function te(e){let t=e.lastIndexOf(`.`);return t===-1?``:e.slice(t+1).toLowerCase()}function x(e){return ee[te(e)]??null}const S=l(c),C=2*1024*1024,w=10*1024*1024;function ne(e){return i(`sha1`).update(n.resolve(e)).digest(`hex`).slice(0,12)}async function T(e){let t=r(e);try{return(await t.status()).current??null}catch{return null}}function E(e,t){return(e+t).trim().includes(`?`)?`untracked`:e===`R`||t===`R`?`renamed`:e===`C`||t===`C`?`copied`:e===`A`||t===`A`?`added`:e===`D`||t===`D`?`deleted`:e===`T`||t===`T`?`type-change`:`modified`}async function D(e,t){switch(t.kind){case`workingTree`:return{workingTree:!0};case`branch`:return{base:await re(e,t.base),head:t.head,workingTree:!1};case`pr`:return{base:`pr/${t.prNumber}/base`,head:`pr/${t.prNumber}/head`,workingTree:!1};case`session`:throw Error(`session context is not backed by git`);case`stash`:throw Error(`stash context is handled by dedicated branches`)}}async function O(e,t){try{return(await e.raw([`rev-parse`,`--verify`,`--quiet`,`${t}^{commit}`])).trim().length>0}catch{return!1}}async function re(e,t){if(t.startsWith(`origin/`))return t;let n=`origin/${t}`;return await O(e,n)?n:t}async function ie(e,t){let n=r(e);if(t.kind===`stash`){let e=t.ref,r=[];if(!await O(n,e))return r;let i=await n.raw([`diff`,`--name-status`,`--find-renames`,`${e}^1`,e]).catch(()=>``),a=A(await n.raw([`diff`,`--numstat`,`${e}^1`,e]).catch(()=>``)),o=k(await n.raw([`diff`,`--raw`,`--find-renames`,`${e}^1`,e]).catch(()=>``));for(let e of i.split(`
|
|
3
|
+
`).filter(Boolean)){let t=e.split(` `),n=t[0],i=`modified`,s,c=t[1];n.startsWith(`A`)?i=`added`:n.startsWith(`D`)?i=`deleted`:n.startsWith(`M`)?i=`modified`:n.startsWith(`R`)?(i=`renamed`,s=t[1],c=t[2]):n.startsWith(`C`)?(i=`copied`,s=t[1],c=t[2]):n.startsWith(`T`)&&(i=`type-change`);let l=a.get(c)??{additions:0,deletions:0,binary:!1};r.push({path:c,oldPath:s,status:i,additions:l.additions,deletions:l.deletions,isBinary:l.binary,contentSig:o.get(c)})}if(await O(n,`${e}^3`)){let t=A(await n.raw([`diff`,`--numstat`,L,`${e}^3`]).catch(()=>``)),i=k(await n.raw([`diff`,`--raw`,L,`${e}^3`]).catch(()=>``));for(let[e,n]of t)r.push({path:e,status:`added`,additions:n.additions,deletions:n.deletions,isBinary:n.binary,contentSig:i.get(e)})}return r}let i=await D(n,t),a=[];if(i.workingTree){let[t,r,i]=await Promise.all([n.status(),n.raw([`diff`,`--numstat`,`HEAD`]).catch(()=>``),n.raw([`diff`,`--find-renames`,`HEAD`]).catch(()=>``)]),o=A(r),s=ae(i),c=[];for(let e of t.files){let t=E(e.index,e.working_dir),n,r=e.path,i=e.path.match(/^(.+) -> (.+)$/);i&&(n=i[1],r=i[2]);let l=o.get(r)??{additions:0,deletions:0,binary:!1};a.push({path:r,oldPath:n,status:t,additions:l.additions,deletions:l.deletions,isBinary:l.binary,contentSig:s.get(r)}),!l.binary&&l.additions===0&&l.deletions===0&&(t===`untracked`||t===`added`)&&c.push({index:a.length-1,filePath:r})}return c.length>0&&await Promise.all(c.map(async({index:t,filePath:n})=>{let r=await oe(e,n);r&&(a[t]={...a[t],additions:r.additions,isBinary:r.binary,contentSig:r.sig??a[t].contentSig})})),a}if(i.base&&i.head){let[e,t]=await Promise.all([O(n,i.base),O(n,i.head)]);if(!e||!t)return a;let[r,o,s]=await Promise.all([n.raw([`diff`,`--name-status`,`--find-renames`,`${i.base}...${i.head}`]),n.raw([`diff`,`--numstat`,`${i.base}...${i.head}`]),n.raw([`diff`,`--raw`,`--find-renames`,`${i.base}...${i.head}`]).catch(()=>``)]),c=A(o),l=k(s);for(let e of r.split(`
|
|
4
|
+
`).filter(Boolean)){let t=e.split(` `),n=t[0],r=`modified`,i,o=t[1];n.startsWith(`A`)?r=`added`:n.startsWith(`D`)?r=`deleted`:n.startsWith(`M`)?r=`modified`:n.startsWith(`R`)?(r=`renamed`,i=t[1],o=t[2]):n.startsWith(`C`)?(r=`copied`,i=t[1],o=t[2]):n.startsWith(`T`)&&(r=`type-change`);let s=c.get(o)??{additions:0,deletions:0,binary:!1};a.push({path:o,oldPath:i,status:r,additions:s.additions,deletions:s.deletions,isBinary:s.binary,contentSig:l.get(o)})}}return a}function k(e){let t=new Map;for(let n of e.split(`
|
|
5
|
+
`).filter(Boolean)){let e=n.indexOf(` `);if(e===-1)continue;let r=n.slice(0,e).split(` `)[3];if(!r||/^0+$/.test(r))continue;let i=n.slice(e+1).split(` `),a=i[i.length-1];t.set(a,r)}return t}function ae(e){let t=new Map,n;for(let r of e.split(`
|
|
6
|
+
`))if(r.startsWith(`diff --git `)){let e=r.indexOf(` b/`);n=e===-1?void 0:r.slice(e+3)}else if(n&&r.startsWith(`index `)){let e=r.match(/^index [0-9a-f]+\.\.([0-9a-f]+)/);e&&!/^0+$/.test(e[1])&&t.set(n,e[1])}return t}function A(e){let t=new Map;for(let n of e.split(`
|
|
7
|
+
`).filter(Boolean)){let[e,r,...i]=n.split(` `),a=i.join(` `),o=e===`-`&&r===`-`;t.set(a,{additions:o?0:Number(e)||0,deletions:o?0:Number(r)||0,binary:o})}return t}async function j(e,t,n,r){try{let i=[`diff`,`--numstat`];return t&&n&&i.push(`${t}...${n}`),i.push(`--`,r),A(await e.raw(i)).get(r)??{additions:0,deletions:0,binary:!1}}catch{return{additions:0,deletions:0,binary:!1}}}async function oe(e,t){try{let r=n.join(e,t),a=await s.stat(r);if(!a.isFile())return null;if(a.size===0)return{additions:0,binary:!1,sig:`empty`};if(a.size>C)return{additions:0,binary:!1,sig:`size:${a.size}`};let o=await s.readFile(r),c=i(`sha1`).update(o).digest(`hex`),l=Math.min(o.length,8192);for(let e=0;e<l;e++)if(o[e]===0)return{additions:0,binary:!0,sig:c};let u=0;for(let e=0;e<o.length;e++)o[e]===10&&u++;return o[o.length-1]!==10&&u++,{additions:u,binary:!1,sig:c}}catch{return null}}async function M(e,t,n){try{return await e.show([`${t}:${n}`])}catch{return``}}async function se(e,t){try{let r=await s.readFile(n.join(e,t));return r.byteLength>C?``:r.toString(`utf8`)}catch{return``}}async function N(e,t,n){try{let{stdout:r}=await S(`git`,[`show`,`${t}:${n}`],{cwd:e,encoding:`buffer`,maxBuffer:w});return r}catch{return null}}async function ce(e,t){try{let r=await s.readFile(n.join(e,t));return r.byteLength>w?null:r}catch{return null}}function P(e,t){return`data:${t};base64,${e.toString(`base64`)}`}async function F(e,t,n,r,i){let a=r?await N(e,r,t):null,o=i?await N(e,i,t):await ce(e,t);return{oldImage:a?P(a,n):void 0,newImage:o?P(o,n):void 0}}function I(e,t){let n=t.endsWith(`
|
|
6
8
|
`),r=n?t.slice(0,-1):t,i=r.length===0?[]:r.split(`
|
|
7
9
|
`);if(i.length===0)return``;let a=[`diff --git a/${e} b/${e}`,`new file mode 100644`,`--- /dev/null`,`+++ b/${e}`,`@@ -0,0 +1,${i.length} @@`,...i.map(e=>`+${e}`)];return n||a.push(`\`),a.join(`
|
|
8
10
|
`)+`
|
|
9
|
-
`}async function
|
|
10
|
-
|
|
11
|
+
`}async function le(e,t,n){let i=r(e),a=await O(i,`${n}^3`)&&await i.raw([`cat-file`,`-e`,`${n}^3:${t}`]).then(()=>!0).catch(()=>!1),o=a?void 0:`${n}^1`,s=a?`${n}^3`:n,c=``,l=o?await M(i,o,t):``,u=await M(i,s,t),d=A(a?await i.raw([`diff`,`--numstat`,L,`${n}^3`,`--`,t]).catch(()=>``):await i.raw([`diff`,`--numstat`,`${n}^1`,n,`--`,t]).catch(()=>``)).get(t)??{additions:0,deletions:0,binary:!1},f=d.binary;a||(c=await i.raw([`diff`,`${n}^1`,n,`--`,t]).catch(()=>``));let p=x(t),m,h;p&&({oldImage:m,newImage:h}=await F(e,t,p,o,s));let g=l.length>0||m!==void 0,_=u.length>0||h!==void 0,v;v=_&&!g?`added`:g&&!_?`deleted`:`modified`;let y=l.length>C||u.length>C,b=f&&p!==null;return a&&!c&&v===`added`&&!f&&p===null&&!y&&u&&(c=I(t,u)),(y||b)&&(l=``,u=``),{file:{path:t,oldPath:void 0,status:v,additions:d.additions,deletions:d.deletions,isBinary:f},patch:c,oldContents:l,newContents:u,truncated:y,oldImage:m,newImage:h}}async function ue(e,t,n){let i=r(e);if(n.kind===`stash`)return le(e,t,n.ref);let a=await D(i,n),o=``,s=``,c=``,l=!1,u=0,d=0,f,p=a.base;if(a.base&&a.head){let e=(await i.raw([`merge-base`,a.base,a.head]).catch(()=>``)).trim();e&&(p=e)}if(a.workingTree){o=await i.diff([`HEAD`,`--`,t]).catch(()=>``),o||=await i.diff([`--`,t]).catch(()=>``),s=await M(i,`HEAD`,t),c=await se(e,t);let n=await j(i,void 0,void 0,t);u=n.additions,d=n.deletions,l=n.binary}else if(a.base&&a.head){let[e,n,r,f]=await Promise.all([i.raw([`diff`,`${a.base}...${a.head}`,`--`,t]).catch(()=>``),M(i,p,t),M(i,a.head,t),j(i,a.base,a.head,t)]);o=e,s=n,c=r,u=f.additions,d=f.deletions,l=f.binary}let m=x(t),h,g;if(m){let n=a.workingTree?`HEAD`:p,r=a.workingTree?void 0:a.head;({oldImage:h,newImage:g}=await F(e,t,m,n,r))}let _=s.length>0||h!==void 0,v=c.length>0||g!==void 0;f=v&&!_?`added`:_&&!v?`deleted`:`modified`;let y=s.length>C||c.length>C,b=l&&m!==null;return a.workingTree&&!o&&f===`added`&&!l&&m===null&&!y&&c&&(o=I(t,c)),{file:{path:t,oldPath:void 0,status:f,additions:u,deletions:d,isBinary:l},patch:o,oldContents:y||b?``:s,newContents:y||b?``:c,truncated:y,oldImage:h,newImage:g}}async function de(e){try{let t=(await e.raw([`symbolic-ref`,`--short`,`refs/remotes/origin/HEAD`])).trim();if(t)return t.replace(/^origin\//,``)}catch{}for(let t of[`origin/main`,`origin/master`,`origin/trunk`,`main`,`master`,`trunk`])try{return await e.raw([`rev-parse`,`--verify`,`--quiet`,`${t}^{commit}`]),t.replace(/^origin\//,``)}catch{}}async function fe(e){return de(r(e))}const L=`4b825dc642cb6eb9a060e54bf8d69288fbee4904`;function pe(e,t){let n=[];for(let r of e){let e=r.files.filter(e=>t.has(e));if(e.length===0)continue;let i=`step-${n.length+1}`,a=new Set(e),o=[];for(let e of r.callouts??[]){if(!a.has(e.file))continue;let t=Math.max(1,Math.floor(e.startLine)),n=Math.max(t,Math.floor(e.endLine??e.startLine));Number.isFinite(t)&&o.push({id:`${i}-c${o.length+1}`,file:e.file,startLine:t,endLine:n,side:e.side===`old`?`old`:`new`,body:e.body??``})}n.push({id:i,title:r.title,body:r.body??``,paths:e,callouts:o})}return n}async function me(e,t,n,i={kind:`workingTree`}){let o=ne(e),s=r(e),c=i.kind===`branch`?i.base:`HEAD`,l=await s.revparse([c]).then(e=>e.trim(),()=>void 0)??await s.revparse([`HEAD`]).then(e=>e.trim(),()=>void 0),u=await T(e).catch(()=>null)??void 0,d=await ie(e,i),f=[],p=0,m=0;for(let t of d){let n=await ue(e,t.path,i);f.push({path:n.file.path,oldPath:n.file.oldPath??t.oldPath,status:n.file.status,additions:n.file.additions,deletions:n.file.deletions,isBinary:n.file.isBinary,patch:n.patch,oldContents:n.oldContents,newContents:n.newContents,truncated:n.truncated,oldImage:n.oldImage,newImage:n.newImage}),p+=n.file.additions,m+=n.file.deletions}let h=pe(t.steps??n?.steps?.map(e=>({title:e.title,body:e.body,files:e.paths,callouts:e.callouts?.map(e=>({file:e.file,startLine:e.startLine,endLine:e.endLine,side:e.side,body:e.body}))}))??[],new Set(f.map(e=>e.path))),g=Date.now();return{id:n?.id??a(),repoId:o,key:t.key??n?.key,name:t.name??n?.name??`Untitled session`,description:t.description??n?.description??``,harness:t.harness??n?.harness??`other`,harnessLabel:t.harnessLabel??n?.harnessLabel,harnessUrl:t.harnessUrl??n?.harnessUrl,branch:u,baseRef:l,createdAt:n?.createdAt??g,updatedAt:g,fileCount:f.length,additions:p,deletions:m,stepCount:h.length,steps:h,files:f}}l(c);function R(e){return n.join(e,`.super-review`,`sessions`)}function z(e,t){return n.join(R(e),`${t}.json`)}function he(e){let{files:t,steps:n,...r}=e;return{...r,stepCount:n?.length??0}}async function B(e){try{let t=await s.readFile(e,`utf8`);return JSON.parse(t)}catch{return null}}async function ge(e){let t=R(e),r;try{r=await s.readdir(t)}catch{return[]}return(await Promise.all(r.filter(e=>e.endsWith(`.json`)).map(e=>B(n.join(t,e))))).filter(e=>e!==null).map(he).sort((e,t)=>t.updatedAt-e.updatedAt)}async function V(e,t){return B(z(e,t))}async function _e(e,t){let r;try{r=await s.readdir(R(e))}catch{return null}for(let i of r){if(!i.endsWith(`.json`))continue;let r=await B(n.join(R(e),i));if(r?.key===t)return r}return null}async function ve(e,t){let n=R(e);await s.mkdir(n,{recursive:!0}),await s.writeFile(z(e,t.id),JSON.stringify(t,null,2),`utf8`)}async function ye(e){await s.rm(R(e),{recursive:!0,force:!0})}const H=_(`comments`,{id:v(`id`).primaryKey(),repo:v(`repo`).notNull(),contextKey:v(`context_key`).notNull(),path:v(`path`).notNull(),side:v(`side`).$type().notNull(),startLine:g(`start_line`).notNull(),endLine:g(`end_line`).notNull(),body:v(`body`).notNull(),author:v(`author`,{mode:`json`}).$type().notNull(),createdAt:g(`created_at`).notNull(),updatedAt:g(`updated_at`).notNull(),resolvedAt:g(`resolved_at`),resolvedBy:v(`resolved_by`,{mode:`json`}).$type(),resolvedSessionId:v(`resolved_session_id`)});function U(){return n.join(u(),`.super-review`)}function be(){return n.join(U(),`comments.db`)}function W(e){return n.resolve(e)}let G=null;function xe(){if(!G){o(U(),{recursive:!0});let e=d({url:`file:${be().replace(/\\/g,`/`)}`});G={client:e,db:f(e),ready:e.executeMultiple(`
|
|
12
|
+
CREATE TABLE IF NOT EXISTS comments (
|
|
13
|
+
id TEXT PRIMARY KEY,
|
|
14
|
+
repo TEXT NOT NULL,
|
|
15
|
+
context_key TEXT NOT NULL,
|
|
16
|
+
path TEXT NOT NULL,
|
|
17
|
+
side TEXT NOT NULL,
|
|
18
|
+
start_line INTEGER NOT NULL,
|
|
19
|
+
end_line INTEGER NOT NULL,
|
|
20
|
+
body TEXT NOT NULL,
|
|
21
|
+
author TEXT NOT NULL,
|
|
22
|
+
created_at INTEGER NOT NULL,
|
|
23
|
+
updated_at INTEGER NOT NULL,
|
|
24
|
+
resolved_at INTEGER,
|
|
25
|
+
resolved_by TEXT,
|
|
26
|
+
resolved_session_id TEXT
|
|
27
|
+
);
|
|
28
|
+
CREATE INDEX IF NOT EXISTS comments_repo_context ON comments (repo, context_key);
|
|
29
|
+
`).then(()=>void 0)}}return G}async function K(){let e=xe();return await e.ready,e.db}function q(e){return{id:e.id,contextKey:e.contextKey,path:e.path,side:e.side,startLine:e.startLine,endLine:e.endLine,body:e.body,author:e.author,createdAt:e.createdAt,updatedAt:e.updatedAt,...e.resolvedAt==null?{}:{resolvedAt:e.resolvedAt},...e.resolvedBy==null?{}:{resolvedBy:e.resolvedBy},...e.resolvedSessionId==null?{}:{resolvedSessionId:e.resolvedSessionId}}}async function Se(e){return(await(await K()).select().from(H).where(h(H.repo,W(e))).orderBy(m(H.updatedAt))).map(q)}async function Ce(e,t){return(await(await K()).select().from(H).where(p(h(H.repo,W(e)),h(H.contextKey,t))).orderBy(m(H.updatedAt))).map(q)}async function J(e,t){let n=await(await K()).select().from(H).where(p(h(H.repo,W(e)),h(H.id,t))).limit(1);return n[0]?q(n[0]):null}async function we(e,t,n,r){let i=await J(e,t);if(!i)return null;let a=await K(),o=Date.now();return await a.update(H).set({resolvedAt:o,resolvedBy:n,resolvedSessionId:r??null,updatedAt:o}).where(p(h(H.repo,W(e)),h(H.id,t))),{...i,resolvedAt:o,resolvedBy:n,...r?{resolvedSessionId:r}:{},updatedAt:o}}async function Te(e,t){let n=await J(e,t);if(!n)return null;let r=await K(),i=Date.now();await r.update(H).set({resolvedAt:null,resolvedBy:null,resolvedSessionId:null,updatedAt:i}).where(p(h(H.repo,W(e)),h(H.id,t)));let{resolvedAt:a,resolvedBy:o,resolvedSessionId:s,...c}=n;return{...c,updatedAt:i}}function Y(e){console.error(`error: ${e}`),process.exit(1)}async function X(e){try{return(await r(e).revparse([`--show-toplevel`])).trim()}catch{Y(`not a git repository: ${e}`)}}const Z=[`claude-code`,`cursor`,`codex`,`opencode`,`copilot`,`other`];function Ee(e){let t;try{t=JSON.parse(e)}catch{Y(`--tour is not valid JSON`)}(typeof t!=`object`||!t)&&Y(`--tour must be a JSON object`);let n=t;return n.steps!==void 0&&(Array.isArray(n.steps)||Y("--tour `steps` must be an array"),n.steps.forEach((e,t)=>{(!e||typeof e.title!=`string`||!e.title.trim())&&Y(`--tour step ${t+1} is missing a non-empty "title"`),(!Array.isArray(e.files)||e.files.length===0||!e.files.every(e=>typeof e==`string`))&&Y(`--tour step ${t+1} ("${e.title}") needs a non-empty "files" array of paths`),e.callouts!==void 0&&(Array.isArray(e.callouts)||Y(`--tour step ${t+1} ("${e.title}") "callouts" must be an array`),e.callouts.forEach((e,n)=>{let r=`step ${t+1} callout ${n+1}`;(!e||typeof e.file!=`string`)&&Y(`--tour ${r} needs a "file" string`),typeof e.startLine!=`number`&&Y(`--tour ${r} needs a numeric "startLine"`),(typeof e.body!=`string`||!e.body.trim())&&Y(`--tour ${r} needs a non-empty "body"`)}))})),n}async function De(e){let t=e.tour?Ee(e.tour):null,r=e.harness??t?.harness;r&&!Z.includes(r)&&Y(`invalid --harness "${r}". Expected one of: ${Z.join(`, `)}`);let i=await X(n.resolve(e.cwd??process.cwd())),a=null;e.id?(a=await V(i,e.id),a||Y(`no session with id "${e.id}" for this repo`)):e.key&&(a=await _e(i,e.key));let o=e.name??t?.name,s=e.description??t?.description;!a&&(!o||!s)&&Y(`--name and --description are required when creating a new session`);let c={key:e.key,name:o,description:s,harness:r||void 0,harnessLabel:e.harnessLabel??t?.harnessLabel,harnessUrl:e.harnessUrl??t?.harnessUrl,steps:t?.steps},l=!!(e.committed||e.base||e.head),u;if(l){let t=e.base??await fe(i);t||Y(`could not determine a base branch to diff against; pass --base <ref>`),u={kind:`branch`,base:t,head:e.head??await T(i)??`HEAD`}}let d=await me(i,c,a,u);if(d.fileCount===0&&Y(l?`no committed changes on ${u.kind===`branch`?u.head:`HEAD`} vs ${u.kind===`branch`?u.base:`base`} to capture`:`no working-tree changes to capture`),t?.steps){let e=new Set(d.files.map(e=>e.path)),n=[...new Set(t.steps.flatMap(e=>[...e.files,...(e.callouts??[]).map(e=>e.file)]).filter(t=>!e.has(t)))];n.length>0&&console.error(`warning: ${n.length} tour path(s) not in the captured changes (dropped):\n`+n.map(e=>` - ${e}`).join(`
|
|
30
|
+
`))}await ve(i,d);let f=a?`updated`:`created`,p=d.stepCount>0?`, ${d.stepCount} tour step(s)`:``;console.log(`${f} session "${d.name}" (${d.id})\n ${d.fileCount} file(s), +${d.additions} −${d.deletions}${p}`)}const Oe=new e(`save`).description(`capture the working tree's current changes (or, with --committed, this branch's committed changes) as a reviewable session`).option(`--key <id>`,`Stable upsert key (your harness conversation/run id). Re-running with the same key UPDATES that session.`).option(`--id <id>`,`Target an existing session by its id (alternative to --key).`).option(`--name <text>`,`Session name (required on first save).`).option(`--description <text>`,`What you changed (required on first save).`).option(`--harness <kind>`,`One of: ${Z.join(`, `)} (default: other).`).option(`--harness-label <text>`,`Freeform harness name (used when --harness other).`).option(`--harness-url <url>`,`Deep link back to this run (resume/permalink).`).option(`--committed`,`Capture this branch diffed against its base (auto-detected default branch) instead of the working tree, so already-committed changes can be documented.`).option(`--base <ref>`,`Base ref to diff against (implies --committed). Defaults to the auto-detected default branch.`).option(`--head <ref>`,`Head ref to diff (implies --committed). Defaults to the current branch.`).option(`--tour <json>`,`Inline JSON describing a guided tour: ordered steps that group related files with commentary, so the reviewer reads the change as a narrative. Flags override the document's top-level name/description/harness.`).option(`--cwd <path>`,`Repo path (default: current directory).`).addHelpText(`after`,`
|
|
11
31
|
Captures a frozen snapshot of the working tree's current changes so they can be
|
|
12
32
|
reviewed as an isolated session in the super-review desktop app.
|
|
13
33
|
|
|
@@ -21,7 +41,20 @@ reviewed as an isolated session in the super-review desktop app.
|
|
|
21
41
|
]
|
|
22
42
|
}
|
|
23
43
|
List files in reading order; any changed file you omit still shows, grouped
|
|
24
|
-
under "Other changes" at the end
|
|
44
|
+
under "Other changes" at the end.
|
|
45
|
+
|
|
46
|
+
Documenting committed changes:
|
|
47
|
+
By default the diff is the working tree (uncommitted edits). To document
|
|
48
|
+
changes you've already committed, pass --committed: it captures this branch
|
|
49
|
+
diffed against its base (auto-detected default branch, e.g. main). Override
|
|
50
|
+
with --base <ref> and/or --head <ref>. Passing --base or --head implies
|
|
51
|
+
--committed.`).action(De);async function ke(e){let t=await X(n.resolve(e.cwd??process.cwd())),r=(await ge(t)).length;await ye(t),console.log(r===0?`no sessions to clear`:`cleared ${r} session${r===1?``:`s`} from .super-review/sessions/`)}const Ae=new e(`clear`).description(`remove all sessions from the repo (.super-review/sessions/)`).option(`--cwd <path>`,`Repo path (default: current directory).`).action(ke),je=new e(`session`).description(`create and manage review sessions`).addHelpText(`after`,`
|
|
25
52
|
Sessions are stored in the repo under .super-review/sessions/, so they can be
|
|
26
53
|
committed and travel with a branch/PR. Run "session clear" to remove them all
|
|
27
|
-
(e.g. before merging) — git then sees the committed manifests as deleted.`).addCommand(
|
|
54
|
+
(e.g. before merging) — git then sees the committed manifests as deleted.`).addCommand(Oe).addCommand(Ae);function Q(e){let t=e.startLine===e.endLine?`L${e.startLine}`:`L${e.startLine}-${e.endLine}`,n=e.resolvedAt?`resolved`:`open`,r=e.body.split(`
|
|
55
|
+
`)[0]?.trim()??``,i=r.length>80?`${r.slice(0,77)}…`:r;return`${e.id} [${n}] ${e.path}:${t} ${i}`}async function Me(e){let t=await X(n.resolve(e.cwd??process.cwd())),r=e.context??`workingTree`,i=e.all?await Se(t):await Ce(t,r);if(e.unresolved&&(i=i.filter(e=>!e.resolvedAt)),e.json){console.log(JSON.stringify(i,null,2));return}if(i.length===0){console.log(e.all?`no comments in this repo`:`no comments in context "${r}"`+(e.unresolved?` (unresolved)`:``));return}for(let e of i)console.log(Q(e))}const Ne=new e(`list`).description(`list local review comments (defaults to the working-tree context)`).option(`--context <key>`,`Diff-context key to list (e.g. "workingTree", "branch:main..feat", "pr:12"). Default: workingTree.`).option(`--all`,`List comments across every context, ignoring --context.`).option(`--unresolved`,`Only show comments that are not yet resolved.`).option(`--json`,`Output the raw comment records as JSON.`).option(`--cwd <path>`,`Repo path (default: current directory).`).action(Me),$=[`claude-code`,`cursor`,`codex`,`opencode`,`copilot`,`other`];async function Pe(e,t){let r=await X(n.resolve(t.cwd??process.cwd()));t.harness&&!$.includes(t.harness)&&Y(`invalid --harness "${t.harness}". Expected one of: ${$.join(`, `)}`);let i=t.harness,a={kind:`agent`,name:t.name??i??`agent`,...i?{harness:i}:{}};t.session&&(await V(r,t.session)||Y(`no session with id "${t.session}" in this repo`));let o=await we(r,e,a,t.session??null);o||Y(`no comment with id "${e}" in this repo`);let s=o.resolvedSessionId?` (linked session ${o.resolvedSessionId})`:``;console.log(`resolved comment ${o.id}${s}`)}async function Fe(e,t){let r=await Te(await X(n.resolve(t.cwd??process.cwd())),e);r||Y(`no comment with id "${e}" in this repo`),console.log(`unresolved comment ${r.id}`)}const Ie=new e(`resolve`).description(`mark a local review comment as resolved`).argument(`<id>`,"The comment id (from `comment list`).").option(`--harness <kind>`,`Resolving agent: one of ${$.join(`, `)}.`).option(`--name <label>`,`Freeform resolver name (overrides the harness label).`).option(`--session <id>`,`Link the session that documents the fix, so the reviewer can jump to it.`).option(`--cwd <path>`,`Repo path (default: current directory).`).action(Pe),Le=new e(`unresolve`).description(`clear a local review comment’s resolved state`).argument(`<id>`,"The comment id (from `comment list`).").option(`--cwd <path>`,`Repo path (default: current directory).`).action(Fe),Re=new e(`comment`).description(`list and resolve local review comments`).addHelpText(`after`,`
|
|
56
|
+
Local review comments are stored in the repo under .super-review/comments/, one
|
|
57
|
+
JSON file per comment, so they travel with a branch/PR like sessions. Humans
|
|
58
|
+
leave comments in the super-review desktop app; agents read them here (use
|
|
59
|
+
"comment list --unresolved" to find work) and mark them resolved with
|
|
60
|
+
"comment resolve <id> --harness <kind> [--session <id>]".`).addCommand(Ne).addCommand(Ie).addCommand(Le);t.name(`super-review`).description(b).version(y).showHelpAfterError().addCommand(je).addCommand(Re).parseAsync(process.argv).catch(e=>{Y(e instanceof Error?e.message:String(e))});export{};
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "super-review",
|
|
3
|
-
"version": "0.0.
|
|
3
|
+
"version": "0.0.3",
|
|
4
4
|
"description": "Document an agent session for review with the super-review desktop app",
|
|
5
5
|
"type": "module",
|
|
6
6
|
"license": "MIT",
|
|
@@ -33,14 +33,16 @@
|
|
|
33
33
|
"access": "public"
|
|
34
34
|
},
|
|
35
35
|
"dependencies": {
|
|
36
|
+
"@libsql/client": "^0.17.3",
|
|
36
37
|
"commander": "^13.1.0",
|
|
38
|
+
"drizzle-orm": "^0.45.2",
|
|
37
39
|
"simple-git": "^3.27.0"
|
|
38
40
|
},
|
|
39
41
|
"devDependencies": {
|
|
40
42
|
"@types/node": "^24.0.0",
|
|
41
43
|
"tsdown": "^0.22.1",
|
|
42
44
|
"typescript": "^5.7.3",
|
|
43
|
-
"@super-review/core": "0.1.
|
|
45
|
+
"@super-review/core": "0.1.4"
|
|
44
46
|
},
|
|
45
47
|
"scripts": {
|
|
46
48
|
"build": "tsdown",
|