@staff0rd/assist 0.294.0 → 0.294.1
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/index.js +8 -10
- package/package.json +1 -1
package/dist/index.js
CHANGED
|
@@ -6,7 +6,7 @@ import { Command } from "commander";
|
|
|
6
6
|
// package.json
|
|
7
7
|
var package_default = {
|
|
8
8
|
name: "@staff0rd/assist",
|
|
9
|
-
version: "0.294.
|
|
9
|
+
version: "0.294.1",
|
|
10
10
|
type: "module",
|
|
11
11
|
main: "dist/index.js",
|
|
12
12
|
bin: {
|
|
@@ -15016,9 +15016,11 @@ function printReviewerFailures(results) {
|
|
|
15016
15016
|
import { existsSync as existsSync38, unlinkSync as unlinkSync14 } from "fs";
|
|
15017
15017
|
|
|
15018
15018
|
// src/commands/review/buildReviewerStdin.ts
|
|
15019
|
-
var REVIEW_PROMPT = `You are acting as a reviewer for a proposed code change made by another engineer. The full review request \u2014 branch, base, changed files, and unified diff \u2014 is in request
|
|
15019
|
+
var REVIEW_PROMPT = `You are acting as a reviewer for a proposed code change made by another engineer. The full review request \u2014 branch, base, changed files, and unified diff \u2014 is in the request file whose absolute path is given below.
|
|
15020
15020
|
|
|
15021
|
-
|
|
15021
|
+
Your working directory is the repository under review, so you can read any file in it for context beyond the diff.
|
|
15022
|
+
|
|
15023
|
+
Read the request file, then produce a thorough code review in Markdown.
|
|
15022
15024
|
|
|
15023
15025
|
## When to flag a finding
|
|
15024
15026
|
|
|
@@ -15054,7 +15056,7 @@ List every finding that the original author would want to know about and fix. Do
|
|
|
15054
15056
|
|
|
15055
15057
|
For each finding include:
|
|
15056
15058
|
- Severity (blocker, major, minor, nit) \u2014 see rubric below
|
|
15057
|
-
- File and line (e.g. \`src/foo.ts:42\`) when the finding is tied to a specific location. Take the line number from the diff's left gutter (its line in the new file); never count lines in request.
|
|
15059
|
+
- File and line (e.g. \`src/foo.ts:42\`) when the finding is tied to a specific location. Take the line number from the diff's left gutter (its line in the new file); never count lines in the request file.
|
|
15058
15060
|
- Impact: what could go wrong, including the conditions under which it manifests
|
|
15059
15061
|
- Recommendation: a concrete change
|
|
15060
15062
|
|
|
@@ -15458,11 +15460,9 @@ function parseCodexEvent(line) {
|
|
|
15458
15460
|
}
|
|
15459
15461
|
|
|
15460
15462
|
// src/commands/review/runCodexReviewer.ts
|
|
15461
|
-
function codexArgs(
|
|
15463
|
+
function codexArgs(outputPath) {
|
|
15462
15464
|
return [
|
|
15463
15465
|
"exec",
|
|
15464
|
-
"--cd",
|
|
15465
|
-
reviewDir,
|
|
15466
15466
|
"--sandbox",
|
|
15467
15467
|
"read-only",
|
|
15468
15468
|
"--json",
|
|
@@ -15476,7 +15476,7 @@ async function runCodexReviewer(spec) {
|
|
|
15476
15476
|
const result = await runStreamingChild({
|
|
15477
15477
|
name: spec.name,
|
|
15478
15478
|
command,
|
|
15479
|
-
args: codexArgs(spec.
|
|
15479
|
+
args: codexArgs(spec.outputPath),
|
|
15480
15480
|
stdin: spec.stdin,
|
|
15481
15481
|
quiet: Boolean(spinner),
|
|
15482
15482
|
onLine: (line) => {
|
|
@@ -15500,7 +15500,6 @@ function resolveCodex(args) {
|
|
|
15500
15500
|
const spinner = args.multi?.create("codex \u2014 starting");
|
|
15501
15501
|
return runCodexReviewer({
|
|
15502
15502
|
name: "codex",
|
|
15503
|
-
reviewDir: args.reviewDir,
|
|
15504
15503
|
stdin: args.stdin,
|
|
15505
15504
|
outputPath: args.codexPath,
|
|
15506
15505
|
spinner
|
|
@@ -15517,7 +15516,6 @@ async function runReviewers(reviewDir, claudePath, codexPath, stdinPrompt, optio
|
|
|
15517
15516
|
multi: options2.multi
|
|
15518
15517
|
});
|
|
15519
15518
|
const codexPromise = resolveCodex({
|
|
15520
|
-
reviewDir,
|
|
15521
15519
|
codexPath,
|
|
15522
15520
|
stdin: stdinPrompt,
|
|
15523
15521
|
plan: options2.codexPlan,
|