mirror of
https://github.com/expo/code-review-cli.git
synced 2026-11-14 17:49:25 -10:00
Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
 Peter Abbondanzo
|
65ffc81014 | ||
|
Brent Vatne
|
92257f9bc9 | ||
|
Brent Vatne
|
047d360b1b |
No files matched your search
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
@@ -9,3 +9,4 @@ build/
|
||||
|
||||
# Claude Code local worktrees
|
||||
.claude/worktrees/
|
||||
.expo-agents/code-review/.runs/
|
||||
@@ -4,7 +4,10 @@ A config-driven, multi-agent AI code reviewer. Specialist agents review a diff i
|
||||
parallel; a coordinator consolidates their findings into one structured review.
|
||||
The same engine runs locally (advisory) and in CI (posts one PR comment). The CLI
|
||||
is the **engine** — each repo supplies its own agents and settings under
|
||||
`.expo-code-review/`, so behavior is configured per-repo, not baked in.
|
||||
`.expo-agents/code-review/`, so behavior is configured per-repo, not baked in.
|
||||
(Repos set up before 0.15 keep their `.expo-code-review/` directory: every command
|
||||
looks for the new name first and falls back to the old one, and `ecr init` reuses
|
||||
an existing legacy directory rather than starting a second one.)
|
||||
|
||||
> **Status: experimental.** Comment-only and non-blocking — it never blocks a merge
|
||||
> and never auto-approves. See [`ROADMAP.md`](./ROADMAP.md).
|
||||
@@ -94,7 +97,7 @@ here ([LLP 0001](./llp/0001-trust-model.principles.md)) is built around that:
|
||||
## Usage
|
||||
|
||||
Run via `npx @expo/code-review-cli <command>` (or the `ecr` / `expo-code-review`
|
||||
binary once installed). On a repo that already has `.expo-code-review/` set up,
|
||||
binary once installed). On a repo that already has `.expo-agents/code-review/` set up,
|
||||
getting model credentials for local runs is one command:
|
||||
|
||||
```bash
|
||||
@@ -108,7 +111,7 @@ package.
|
||||
### First-time setup
|
||||
|
||||
```bash
|
||||
# 1. Scaffold .expo-code-review/ + a CI workflow (--no-workflow to skip)
|
||||
# 1. Scaffold .expo-agents/code-review/ + a CI workflow (--no-workflow to skip)
|
||||
npx @expo/code-review-cli init
|
||||
# 2. Get model credentials — guided; prints the export lines for your shell config
|
||||
npx @expo/code-review-cli setup-auth
|
||||
@@ -142,7 +145,7 @@ ecr review --pr 123
|
||||
ecr review --pr 123 --post
|
||||
# Preview once, save the exact result, and post it later without another model run
|
||||
ecr review --repo owner/repo --pr 123 --save-review --json
|
||||
ecr post-review --artifact .expo-code-review/.runs/deferred/<artifact>.json --repo owner/repo --pr 123
|
||||
ecr post-review --artifact .expo-agents/code-review/.runs/deferred/<artifact>.json --repo owner/repo --pr 123
|
||||
```
|
||||
|
||||
Options (most to least common):
|
||||
@@ -178,7 +181,7 @@ is a ready example to adapt.
|
||||
|
||||
| Command | What it does |
|
||||
| --- | --- |
|
||||
| `ecr init [--no-workflow] [--force]` | Scaffold `.expo-code-review/` (config, agents, prompts) + a CI workflow. |
|
||||
| `ecr init [--no-workflow] [--force]` | Scaffold `.expo-agents/code-review/` (config, agents, prompts) + a CI workflow. |
|
||||
| `ecr init --monorepo` | …and add a `routing.jsonc` routing manifest (one default scope). |
|
||||
| `ecr init --scope <dir>` | Scaffold a per-team scope under `<dir>` and register it in the manifest. |
|
||||
| `ecr setup-auth [--yes]` | Walk through getting model credentials for local runs (ChatGPT/Claude sign-in and/or API keys), printing the `export` lines for your shell config. |
|
||||
@@ -215,7 +218,7 @@ Markdown, `//` in JSONC):
|
||||
|
||||
A target is a file, a `dir/`, `glob:<pattern>`, `file#symbol`, or `doc.md#heading` —
|
||||
never a line number (line numbers rot without any signal). The check is strict on
|
||||
purpose: any backticked token in `.expo-code-review/` that names something that
|
||||
purpose: any backticked token in `.expo-agents/code-review/` that names something that
|
||||
exists in the repo **must** be a ref, because stale citations are exactly the ones
|
||||
nobody annotated. Mark the rare false positive once with
|
||||
`<!-- @ref-ignore knex.raw() -->`. Refs are always repo-root-relative, including in
|
||||
@@ -291,18 +294,18 @@ and no locking.
|
||||
|
||||
```
|
||||
your-monorepo/
|
||||
.expo-code-review/
|
||||
.expo-agents/code-review/
|
||||
routing.jsonc # the manifest — infra-owned, ordered scope list + locked defaults
|
||||
config.jsonc # the default/root scope; the ONLY place auth/tokenEnv lives
|
||||
shared.md coordinator.md agents/
|
||||
apps/
|
||||
api/.expo-code-review/{config.jsonc(NO auth),coordinator.md,agents/} # api team
|
||||
web/.expo-code-review/{config.jsonc(NO auth),coordinator.md,agents/} # web team
|
||||
api/.expo-agents/code-review/{config.jsonc(NO auth),coordinator.md,agents/} # api team
|
||||
web/.expo-agents/code-review/{config.jsonc(NO auth),coordinator.md,agents/} # web team
|
||||
.github/workflows/expo-code-review.yml # unchanged shape: one workflow, one `ecr ci`
|
||||
```
|
||||
|
||||
```jsonc
|
||||
// .expo-code-review/routing.jsonc
|
||||
// .expo-agents/code-review/routing.jsonc
|
||||
{
|
||||
// Central guardrails every scope inherits and CANNOT override.
|
||||
"defaults": {
|
||||
@@ -322,7 +325,7 @@ your-monorepo/
|
||||
```
|
||||
|
||||
```jsonc
|
||||
// apps/api/.expo-code-review/config.jsonc (the api team owns this)
|
||||
// apps/api/.expo-agents/code-review/config.jsonc (the api team owns this)
|
||||
{
|
||||
// NO "auth" block — locked centrally; a tokenEnv here is rejected by loader + CI guard.
|
||||
"model": "anthropic/claude-sonnet-5",
|
||||
@@ -378,8 +381,8 @@ Enforced in code and by an independent CI guard step, not by convention:
|
||||
itself, so even a custom workflow that checks out the head still gets
|
||||
base-commit configuration.
|
||||
|
||||
Enforce ownership with CODEOWNERS: `/.expo-code-review/routing.jsonc @your-infra`,
|
||||
`/apps/api/.expo-code-review/ @your-api-team`. Full detail:
|
||||
Enforce ownership with CODEOWNERS: `/.expo-agents/code-review/routing.jsonc @your-infra`,
|
||||
`/apps/api/.expo-agents/code-review/ @your-api-team`. Full detail:
|
||||
[LLP 0006](./llp/0006-config-schema-loading-routing.explainer.md) (routing) and
|
||||
[LLP 0001](./llp/0001-trust-model.principles.md) (trust model).
|
||||
|
||||
@@ -396,7 +399,7 @@ Enforce ownership with CODEOWNERS: `/.expo-code-review/routing.jsonc @your-infra
|
||||
- **Chunking** — small PRs run in a single pass; large PRs are split into chunks
|
||||
bounded by changed lines, plus one combined **cross-cutting pass** that looks
|
||||
for issues spanning multiple changed files across every agent's concern.
|
||||
- **Agents** — every `.md` file in `.expo-code-review/agents/` is an agent. They
|
||||
- **Agents** — every `.md` file in `.expo-agents/code-review/agents/` is an agent. They
|
||||
run in parallel with read-only repo tools (`read`/`grep`/`glob`/`list`).
|
||||
- **Coordinator** — a single pass that dedupes, re-judges severity, and produces
|
||||
the final `{ decision, findings, summary }`.
|
||||
@@ -426,7 +429,7 @@ Every run reports what it spent and how much was served from the prompt cache, i
|
||||
three places: one `Token usage — …` line in the job log, a per-pass table + cache
|
||||
hit rate in the GitHub Actions step summary (which also preserves each run's posted
|
||||
comment, since the PR comment is updated in place), and one JSON line per run in
|
||||
`.expo-code-review/.runs/reviews.jsonl` (uploaded as a CI artifact) with per-pass
|
||||
`.expo-agents/code-review/.runs/reviews.jsonl` (uploaded as a CI artifact) with per-pass
|
||||
tokens, raw per-agent findings, bounded reviewer traces, and coverage notes.
|
||||
|
||||
Each reviewer can also return a compact trace (up to three concrete checks and two
|
||||
@@ -452,10 +455,10 @@ cacheable size (~1–4K tokens) show `cache read 0` — expected, not a bug.
|
||||
</details>
|
||||
|
||||
<details>
|
||||
<summary><b>Configuration — <code>.expo-code-review/</code></b></summary>
|
||||
<summary><b>Configuration — <code>.expo-agents/code-review/</code></b></summary>
|
||||
|
||||
```
|
||||
.expo-code-review/
|
||||
.expo-agents/code-review/
|
||||
config.jsonc # model, policy, noise, auth, break-glass, comment tag
|
||||
shared.md # instructions prepended to every agent (optional)
|
||||
coordinator.md # the consolidation prompt (required)
|
||||
@@ -480,7 +483,7 @@ temperature: 0.1
|
||||
```
|
||||
|
||||
For a real-world example, see eas-cli's
|
||||
[`.expo-code-review/`](https://github.com/expo/eas-cli/tree/main/.expo-code-review)
|
||||
[`.expo-agents/code-review/`](https://github.com/expo/eas-cli/tree/main/.expo-code-review)
|
||||
— correctness/security/consistency agents, a stronger model for security, and
|
||||
per-repo `noise.additionalIgnores`.
|
||||
|
||||
@@ -718,7 +721,7 @@ variables: `ATLANTIS_BOT_LOGIN` (the Atlantis bot's comment login, e.g.
|
||||
<details>
|
||||
<summary><b>Run logs</b></summary>
|
||||
|
||||
Each run appends a JSON line to `.expo-code-review/.runs/reviews.jsonl` with the
|
||||
Each run appends a JSON line to `.expo-agents/code-review/.runs/reviews.jsonl` with the
|
||||
inputs, decision, finding count, duration, per-agent cost, and aggregate token
|
||||
usage (incl. prompt-cache read/write counts) — for auditing and measuring
|
||||
cost/latency/cache reuse over time. It also records the same bounded `reviewTrace`
|
||||
@@ -727,7 +730,7 @@ one-line summary to the terminal / CI job log at the end of each run, so cache r
|
||||
is visible even in CI (where the run log is ephemeral).
|
||||
|
||||
`ecr review --save-review` additionally writes a versioned artifact under
|
||||
`.expo-code-review/.runs/deferred/` with owner-only permissions. It contains the
|
||||
`.expo-agents/code-review/.runs/deferred/` with owner-only permissions. It contains the
|
||||
verified final review and bounded feedback metadata, but no credential. `ecr
|
||||
post-review` schema-validates it and refuses to post if its explicit repo/PR, live
|
||||
head commit, or local comment-policy fingerprint no longer matches.
|
||||
@@ -760,20 +763,25 @@ fallback key; `ecr doctor` diagnoses setup.
|
||||
} }
|
||||
```
|
||||
|
||||
- **Meta / Muse Spark 1.2 (public Model API).** Set the default model and any
|
||||
model-pinned agent frontmatter to `meta/muse-spark-1.2`. ECR supplies the fixed
|
||||
- **Meta / Muse Spark 1.3 (public Model API).** Set the default model and any
|
||||
model-pinned agent frontmatter to `meta/muse-spark-1.3`. ECR supplies the fixed
|
||||
public Responses endpoint and the `@ai-sdk/openai` adapter; the generic
|
||||
Chat-Completions compatibility adapter is intentionally not used because it
|
||||
loses Muse's reasoning continuity across tool turns. Standard and Contributor
|
||||
model ids are supported.
|
||||
|
||||
```jsonc
|
||||
"model": "meta/muse-spark-1.2",
|
||||
"model": "meta/muse-spark-1.3",
|
||||
"auth": { "providers": {
|
||||
"meta": { "mode": "api-key", "tokenEnv": "META_API_KEY" }
|
||||
} }
|
||||
```
|
||||
|
||||
Standard and Contributor 1.2/1.3 model ids have known capability metadata. Any
|
||||
future model explicitly named as `meta/<model>` is still routed to the fixed Meta
|
||||
endpoint with neutral metadata, so a renamed family such as `meta/muse-foo` does
|
||||
not require an ECR release; Meta validates an unknown id on its first request.
|
||||
|
||||
Create the key in the [Meta AI developer portal](https://developer.meta.com/ai/).
|
||||
Locally, export it as `META_API_KEY`. In each review-running workflow, replace
|
||||
the Anthropic credential line with
|
||||
|
||||
@@ -109,12 +109,15 @@ the Responses API and preserve encrypted reasoning across tool turns. The generi
|
||||
Completions path loses that continuity [observed:
|
||||
`opencode.ts` `META_MODEL_API_BASE_URL` / `META_MUSE_MODELS` / `buildOpencodeConfig`].
|
||||
|
||||
The declared model catalog is an allowlist (`muse-spark-1.2` and
|
||||
`muse-spark-1.2-contributor`), not a reflection of arbitrary config strings. This keeps
|
||||
the existing preflight honest: a misspelled or not-yet-supported Meta model remains
|
||||
absent from the server's provider model list and fails once before any pass. Both models
|
||||
declare their public context/output limits, multimodal inputs, high reasoning, and
|
||||
encrypted reasoning inclusion [observed: `opencode.ts` `META_MUSE_MODELS`].
|
||||
Provider selection is forward-compatible at the provider boundary, not coupled to the
|
||||
current Muse family name: every referenced non-empty `meta/<model>` id is declared on
|
||||
the synthesized Meta provider. Known Muse Spark 1.2 and 1.3 standard/Contributor ids
|
||||
receive their published limits, multimodal inputs, high reasoning, and encrypted
|
||||
reasoning inclusion from `META_MUSE_MODELS`; a future id such as `meta/muse-foo` is
|
||||
declared with neutral metadata rather than guessed capabilities. This deliberately
|
||||
moves validation of an unknown-but-well-formed Meta id from OpenCode's local preflight
|
||||
to its first Meta request, in exchange for new Meta names working without an ECR
|
||||
release [observed: `opencode.ts` `META_MUSE_MODELS` / `buildOpencodeConfig`].
|
||||
|
||||
## Claude Code CLI Containment
|
||||
|
||||
|
||||
@@ -184,8 +184,7 @@ hardcoded to a read-only tool set (`read`/`grep`/`glob`/`list`) with no schema k
|
||||
write/bash [observed: `load.ts:126-155`, `load.ts:22-23`].
|
||||
|
||||
`resolveConfigDir` is the escape hatch (graft 1): precedence is explicit param → `ECR_CONFIG_DIR` env
|
||||
→ the repo's default `.expo-code-review/`, and with neither override the result is byte-identical to
|
||||
the default path [observed: `load.ts:29-41`]. `hasConfig` resolves the config dir the same way, so
|
||||
→ the repo's default setup dir [observed: `load.ts`]. **The default setup dir has two spellings** (0.15.0): `.expo-agents/code-review/` is the home — one `.expo-agents/` directory holds every Expo agent tool's per-repo files (this tool's and `@expo/verify`'s) instead of one root dot-directory per tool — and the pre-0.15 `.expo-code-review/` stays fully supported. `configDirFor(base)` returns whichever exists, new name first, and the new name when neither does; it is used for the root and for every scope directory, `ecr init` writes into an existing legacy directory rather than doubling it, the config-file sweep (`isConfigDirPath`) and the reference index accept both, and the CI templates upload `.runs/reviews.jsonl` from either location. `.agents/` was rejected as the shared name because Codex, Cursor, Gemini CLI and Copilot already scan `.agents/skills/`. `hasConfig` resolves the config dir the same way, so
|
||||
`doctor`'s "no config" diagnostic can never disagree with what the loader actually reads
|
||||
[observed: `load.ts:55-60`]. The override designates an alternate ROOT config dir, so `config.jsonc`
|
||||
and `routing.jsonc` always travel together: `loadRoutingManifest` reads through the same
|
||||
|
||||
@@ -56,7 +56,7 @@ The three workflows share a posture built entirely around one fact: a reviewer j
|
||||
**Concurrency defaults by auth mode.** The commented `chunk.concurrency` default is 6 for an API key and 3 for a subscription (OAuth) credential, "one account handles many parallel streams poorly, and several PRs may review on the same credential at once" [observed: `templates/config.jsonc:28-30`].
|
||||
|
||||
**Muse Spark is opt-in without weakening the default secret surface.** The root config
|
||||
documents `meta/muse-spark-1.2` plus a `META_API_KEY` auth entry, and each
|
||||
documents `meta/muse-spark-1.3` plus a `META_API_KEY` auth entry, and each
|
||||
review-running workflow shows the corresponding secret line commented out. It is not
|
||||
forwarded by the default scaffold: adopters replace the Anthropic line and set
|
||||
`ECR_EXPECTED_TOKEN_ENV=META_API_KEY`, so the model process receives only the
|
||||
|
||||
@@ -0,0 +1,46 @@
|
||||
import { describe, expect, test } from "bun:test";
|
||||
import { mkdir, mkdtemp, writeFile } from "node:fs/promises";
|
||||
import { tmpdir } from "node:os";
|
||||
import path from "node:path";
|
||||
import {
|
||||
CONFIG_DIRNAME,
|
||||
LEGACY_CONFIG_DIRNAME,
|
||||
configDirFor,
|
||||
isConfigDirPath,
|
||||
resolveConfigDir,
|
||||
} from "../config/load.js";
|
||||
|
||||
describe("setup dir: .expo-agents/code-review with the legacy .expo-code-review fallback", () => {
|
||||
test("a fresh repo resolves to the new name", async () => {
|
||||
const root = await mkdtemp(path.join(tmpdir(), "ecr-dir-"));
|
||||
expect(configDirFor(root)).toBe(path.join(root, CONFIG_DIRNAME));
|
||||
});
|
||||
|
||||
test("a pre-0.15 repo keeps using .expo-code-review unchanged", async () => {
|
||||
const root = await mkdtemp(path.join(tmpdir(), "ecr-dir-"));
|
||||
await mkdir(path.join(root, LEGACY_CONFIG_DIRNAME), { recursive: true });
|
||||
expect(configDirFor(root)).toBe(path.join(root, LEGACY_CONFIG_DIRNAME));
|
||||
expect(resolveConfigDir(root)).toBe(path.join(root, LEGACY_CONFIG_DIRNAME));
|
||||
});
|
||||
|
||||
test("when both exist the new name wins", async () => {
|
||||
const root = await mkdtemp(path.join(tmpdir(), "ecr-dir-"));
|
||||
await mkdir(path.join(root, LEGACY_CONFIG_DIRNAME), { recursive: true });
|
||||
await mkdir(path.join(root, CONFIG_DIRNAME), { recursive: true });
|
||||
await writeFile(path.join(root, CONFIG_DIRNAME, "config.jsonc"), "{}");
|
||||
expect(configDirFor(root)).toBe(path.join(root, CONFIG_DIRNAME));
|
||||
});
|
||||
|
||||
test("an explicit override still beats both", async () => {
|
||||
const root = await mkdtemp(path.join(tmpdir(), "ecr-dir-"));
|
||||
await mkdir(path.join(root, LEGACY_CONFIG_DIRNAME), { recursive: true });
|
||||
expect(resolveConfigDir(root, "custom")).toBe(path.join(root, "custom"));
|
||||
});
|
||||
|
||||
test("isConfigDirPath recognizes either spelling, nowhere else", () => {
|
||||
expect(isConfigDirPath("/r/.expo-code-review")).toBe(true);
|
||||
expect(isConfigDirPath("/r/apps/api/.expo-agents/code-review")).toBe(true);
|
||||
expect(isConfigDirPath("/r/.expo-agents")).toBe(false);
|
||||
expect(isConfigDirPath("/r/code-review")).toBe(false);
|
||||
});
|
||||
});
|
||||
@@ -503,7 +503,9 @@ test("setup problems are labelled by setup-dir position when the config lives ou
|
||||
setupDirs: [path.join(trustedBase, ".expo-code-review")],
|
||||
});
|
||||
expect(report.problems).toHaveLength(1);
|
||||
expect(report.problems[0]!.file).toBe(path.join(".expo-code-review", "agents", "security.md"));
|
||||
expect(report.problems[0]!.file).toBe(
|
||||
path.join(".expo-agents", "code-review", "agents", "security.md"),
|
||||
);
|
||||
});
|
||||
|
||||
test("citedPathsTouchedBy reports cited paths a PR changed, dirs included", async () => {
|
||||
|
||||
@@ -50,6 +50,36 @@ test("run with input: timeout kills the child and resolves timedOut (no throw)",
|
||||
expect(Date.now() - started).toBeLessThan(10_000);
|
||||
});
|
||||
|
||||
/**
|
||||
* Wait until `pid` no longer names a LIVE process. `kill(pid, 0)` alone is racy
|
||||
* here: it still succeeds while the killed grandchild is a zombie awaiting init's
|
||||
* reap (the direct `sh` dies first, the orphaned `sleep` reparents, and reaping is
|
||||
* asynchronous) — which is exactly what flaked on loaded CI runners. A zombie is
|
||||
* dead for this test's purpose, so a `ps` state of `Z` (or no row) counts as gone.
|
||||
* A process that genuinely survived the group kill (`sleep 30`) outlives the whole
|
||||
* poll window and still fails the assertion.
|
||||
*/
|
||||
async function processGone(pid: number, timeoutMs = 5000): Promise<boolean> {
|
||||
const started = Date.now();
|
||||
for (;;) {
|
||||
try {
|
||||
process.kill(pid, 0);
|
||||
} catch {
|
||||
return true; // ESRCH: fully gone
|
||||
}
|
||||
const state = (
|
||||
await run("ps", ["-o", "state=", "-p", String(pid)], { check: false })
|
||||
).stdout.trim();
|
||||
if (state === "" || state.startsWith("Z")) {
|
||||
return true; // reaped between the two checks, or dead-but-unreaped
|
||||
}
|
||||
if (Date.now() - started >= timeoutMs) {
|
||||
return false;
|
||||
}
|
||||
await new Promise((resolve) => setTimeout(resolve, 50));
|
||||
}
|
||||
}
|
||||
|
||||
test("run with input: timeout kills the whole process group, not just the direct child", async () => {
|
||||
const dir = await mkdtemp(join(tmpdir(), "ecr-exec-test-"));
|
||||
const pidFile = join(dir, "grandchild.pid");
|
||||
@@ -61,13 +91,14 @@ test("run with input: timeout kills the whole process group, not just the direct
|
||||
});
|
||||
expect(result.timedOut).toBe(true);
|
||||
const grandchildPid = Number((await readFile(pidFile, "utf8")).trim());
|
||||
// If only the direct `sh` had been signaled, this detached grandchild `sleep`
|
||||
// would still be alive; process.kill throws ESRCH once the whole group is gone.
|
||||
expect(() => process.kill(grandchildPid, 0)).toThrow();
|
||||
// If only the direct `sh` had been signaled, this grandchild `sleep 30` would
|
||||
// still be alive well past the poll window; the group kill leaves it dead (or
|
||||
// momentarily zombie) within milliseconds.
|
||||
expect(await processGone(grandchildPid)).toBe(true);
|
||||
} finally {
|
||||
await rm(dir, { recursive: true, force: true });
|
||||
}
|
||||
});
|
||||
}, 10_000);
|
||||
|
||||
test("run with input: escalates to SIGKILL after the grace period when the child traps SIGTERM", async () => {
|
||||
const started = Date.now();
|
||||
|
||||
@@ -342,10 +342,10 @@ test("an upstream-alias auth entry synthesizes a provider block with exactly the
|
||||
expect(opencode.provider?.openai).toBeUndefined();
|
||||
});
|
||||
|
||||
test("Meta auth synthesizes the public Muse Responses provider without embedding its key", () => {
|
||||
test("Meta auth synthesizes the public Muse 1.3 Responses provider without embedding its key", () => {
|
||||
const config = configWith({
|
||||
agents: [{ id: "correctness", model: "meta/muse-spark-1.2" }],
|
||||
coordinatorModel: "meta/muse-spark-1.2",
|
||||
agents: [{ id: "correctness", model: "meta/muse-spark-1.3" }],
|
||||
coordinatorModel: "meta/muse-spark-1.3-contributor",
|
||||
auth: [{ mode: "api-key", provider: "meta", tokenEnv: "META_API_KEY" }],
|
||||
});
|
||||
const opencode = buildOpencodeConfig(config) as {
|
||||
@@ -354,7 +354,14 @@ test("Meta auth synthesizes the public Muse Responses provider without embedding
|
||||
{
|
||||
npm: string;
|
||||
options: { baseURL: string; apiKey: string };
|
||||
models: Record<string, { reasoning?: boolean; options?: Record<string, unknown> }>;
|
||||
models: Record<
|
||||
string,
|
||||
{
|
||||
reasoning?: boolean;
|
||||
limit?: { context: number; output: number };
|
||||
options?: Record<string, unknown>;
|
||||
}
|
||||
>;
|
||||
}
|
||||
>;
|
||||
};
|
||||
@@ -365,12 +372,17 @@ test("Meta auth synthesizes the public Muse Responses provider without embedding
|
||||
baseURL: META_MODEL_API_BASE_URL,
|
||||
apiKey: "{env:META_API_KEY}",
|
||||
});
|
||||
expect(Object.keys(meta!.models)).toEqual(["muse-spark-1.2"]);
|
||||
expect(meta!.models["muse-spark-1.2"]?.reasoning).toBe(true);
|
||||
expect(meta!.models["muse-spark-1.2"]?.options).toMatchObject({
|
||||
expect(Object.keys(meta!.models)).toEqual(["muse-spark-1.3", "muse-spark-1.3-contributor"]);
|
||||
expect(meta!.models["muse-spark-1.3"]?.reasoning).toBe(true);
|
||||
expect(meta!.models["muse-spark-1.3"]?.limit).toEqual({
|
||||
context: 1_048_576,
|
||||
output: 1_048_576,
|
||||
});
|
||||
expect(meta!.models["muse-spark-1.3"]?.options).toMatchObject({
|
||||
reasoningEffort: "high",
|
||||
include: ["reasoning.encrypted_content"],
|
||||
});
|
||||
expect(meta!.models["muse-spark-1.3-contributor"]?.reasoning).toBe(true);
|
||||
});
|
||||
|
||||
test("a Meta-named upstream alias does not redirect the upstream credential to Meta", () => {
|
||||
@@ -400,25 +412,22 @@ test("a Meta-named upstream alias does not redirect the upstream credential to M
|
||||
expect(Object.keys(alias!.models)).toEqual(["muse-spark-1.2"]);
|
||||
});
|
||||
|
||||
test("Meta provider does not register an unknown Muse id, so preflight can reject it", () => {
|
||||
test("Meta provider registers future model families neutrally and only for the meta prefix", () => {
|
||||
const config = configWith({
|
||||
agents: [{ id: "correctness", model: "meta/muse-spark-typo" }],
|
||||
coordinatorModel: "meta/muse-spark-typo",
|
||||
agents: [
|
||||
{ id: "correctness", model: "meta/muse-foo" },
|
||||
{ id: "security", model: "openai/muse-foo" },
|
||||
],
|
||||
coordinatorModel: "meta/a-future-family-2",
|
||||
auth: [{ mode: "api-key", provider: "meta", tokenEnv: "META_API_KEY" }],
|
||||
});
|
||||
const opencode = buildOpencodeConfig(config) as {
|
||||
provider?: Record<string, { models: Record<string, unknown> }>;
|
||||
};
|
||||
expect(opencode.provider?.meta?.models).toEqual({});
|
||||
expect(
|
||||
findUnknownModels(["meta/muse-spark-typo"], { meta: ["muse-spark-1.2"] }, ["meta"]),
|
||||
).toEqual([
|
||||
{
|
||||
model: "meta/muse-spark-typo",
|
||||
reason: "model",
|
||||
suggestions: ["muse-spark-1.2"],
|
||||
},
|
||||
]);
|
||||
expect(opencode.provider?.meta?.models).toEqual({
|
||||
"muse-foo": {},
|
||||
"a-future-family-2": {},
|
||||
});
|
||||
});
|
||||
|
||||
test("buildOpencodeConfig registers the no-tools stack verifier (empty tool list)", () => {
|
||||
|
||||
+2
-2
@@ -4,7 +4,7 @@ import { readFile } from "node:fs/promises";
|
||||
import path from "node:path";
|
||||
|
||||
import {
|
||||
CONFIG_DIRNAME,
|
||||
configDirFor,
|
||||
hasScopeConfig,
|
||||
loadAuthFromRoot,
|
||||
loadReviewConfig,
|
||||
@@ -495,7 +495,7 @@ function resolveStackConfirm(
|
||||
* `.expo-code-review/.runs/reviews.jsonl` from the workspace as an artifact.
|
||||
*/
|
||||
function workspaceRunsDir(cwd: string): string {
|
||||
return path.join(cwd, CONFIG_DIRNAME, ".runs");
|
||||
return path.join(configDirFor(cwd), ".runs");
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -4,6 +4,7 @@ import {
|
||||
loadScopeConfig,
|
||||
loadAuthFromRoot,
|
||||
hasConfig,
|
||||
CONFIG_DIRNAME,
|
||||
resolveConfigDir,
|
||||
tokenEnvMismatch,
|
||||
} from "../config/load.js";
|
||||
@@ -246,7 +247,7 @@ export async function doctorCommand(argv: string[] = []): Promise<void> {
|
||||
}
|
||||
|
||||
if (!hasConfig(root)) {
|
||||
line(false, `no ${".expo-code-review"}/config.jsonc (run \`ecr init\`)`);
|
||||
line(false, `no ${CONFIG_DIRNAME}/config.jsonc (run \`ecr init\`)`);
|
||||
} else {
|
||||
try {
|
||||
// Reuse the engine-detection peek above; re-load only if that failed so
|
||||
@@ -479,9 +480,7 @@ async function listScopes(root: string): Promise<void> {
|
||||
return;
|
||||
}
|
||||
if (!manifest) {
|
||||
process.stdout.write(
|
||||
`No ${".expo-code-review"}/routing.jsonc — run \`ecr init --monorepo\`.\n`,
|
||||
);
|
||||
process.stdout.write(`No ${CONFIG_DIRNAME}/routing.jsonc — run \`ecr init --monorepo\`.\n`);
|
||||
process.exitCode = 0;
|
||||
return;
|
||||
}
|
||||
|
||||
@@ -4,7 +4,7 @@ import { existsSync } from "node:fs";
|
||||
import path from "node:path";
|
||||
import { fileURLToPath } from "node:url";
|
||||
|
||||
import { CONFIG_DIRNAME } from "../config/load.js";
|
||||
import { CONFIG_DIRNAME, configDirFor } from "../config/load.js";
|
||||
import { ROUTING_FILENAME } from "../config/routing.js";
|
||||
import { RoutingScopeSchema, type RoutingScope } from "../config/schema.js";
|
||||
import { FORBIDDEN_TOKEN_ENVS } from "../core/auth.js";
|
||||
@@ -13,7 +13,7 @@ import { errorMessage } from "../core/util.js";
|
||||
|
||||
const TEMPLATES_DIR = fileURLToPath(new URL("../../templates/", import.meta.url));
|
||||
|
||||
const USAGE = `ecr init — scaffold .expo-code-review/ in the current repo
|
||||
const USAGE = `ecr init — scaffold .expo-agents/code-review/ in the current repo (a legacy .expo-code-review/ is reused)
|
||||
|
||||
Usage:
|
||||
ecr init [--no-workflow] [--force] Scaffold the root config (+ CI workflow)
|
||||
@@ -22,8 +22,8 @@ Usage:
|
||||
register it in the root routing.jsonc
|
||||
|
||||
Options:
|
||||
--monorepo Also write .expo-code-review/routing.jsonc (routing manifest)
|
||||
--scope <dir> Scaffold <dir>/.expo-code-review/ (no auth) + add a scope entry
|
||||
--monorepo Also write routing.jsonc in the setup dir (routing manifest)
|
||||
--scope <dir> Scaffold <dir>/.expo-agents/code-review/ (no auth) + add a scope entry
|
||||
--no-workflow Skip writing the CI workflows (review, command, and dismiss
|
||||
under .github/workflows/)
|
||||
--token-env <name[,name…]>
|
||||
@@ -83,7 +83,7 @@ async function scaffold(argv: string[]): Promise<void> {
|
||||
}
|
||||
|
||||
const root = (await repoRoot()) ?? process.cwd();
|
||||
const configDir = path.join(root, CONFIG_DIRNAME);
|
||||
const configDir = configDirFor(root); // an existing legacy .expo-code-review/ is kept, never doubled
|
||||
|
||||
// A non-default --token-env only takes effect through the review workflows,
|
||||
// but existing workflow files are skipped (not rewritten) without --force —
|
||||
@@ -286,7 +286,7 @@ async function scaffoldScope(argv: string[], scopeDirRaw: string): Promise<void>
|
||||
);
|
||||
}
|
||||
|
||||
const configDir = path.join(root, scopeDir, CONFIG_DIRNAME);
|
||||
const configDir = configDirFor(path.join(root, scopeDir));
|
||||
await mkdir(configDir, { recursive: true });
|
||||
|
||||
const created: string[] = [];
|
||||
|
||||
@@ -2,7 +2,13 @@
|
||||
import { readdir, readFile } from "node:fs/promises";
|
||||
import path from "node:path";
|
||||
|
||||
import { CONFIG_DIRNAME, stripJsonComments, stripTrailingCommas } from "../config/load.js";
|
||||
import {
|
||||
CONFIG_DIRNAME,
|
||||
configDirFor,
|
||||
isConfigDirPath,
|
||||
stripJsonComments,
|
||||
stripTrailingCommas,
|
||||
} from "../config/load.js";
|
||||
import { ROUTING_FILENAME } from "../config/routing.js";
|
||||
import { repoRoot } from "../core/exec.js";
|
||||
import { errorMessage } from "../core/util.js";
|
||||
@@ -76,11 +82,7 @@ async function discoverConfigFiles(root: string): Promise<string[]> {
|
||||
continue;
|
||||
}
|
||||
await walk(path.join(dir, entry.name));
|
||||
} else if (
|
||||
entry.isFile() &&
|
||||
path.basename(dir) === CONFIG_DIRNAME &&
|
||||
CONFIG_FILENAMES.has(entry.name)
|
||||
) {
|
||||
} else if (entry.isFile() && isConfigDirPath(dir) && CONFIG_FILENAMES.has(entry.name)) {
|
||||
found.push(path.join(dir, entry.name));
|
||||
}
|
||||
}
|
||||
@@ -145,7 +147,7 @@ export async function verifyConfig(
|
||||
options: { expected?: string } = {},
|
||||
): Promise<VerifyResult> {
|
||||
const findings: VerifyFinding[] = [];
|
||||
const rootConfigDir = path.join(root, CONFIG_DIRNAME);
|
||||
const rootConfigDir = configDirFor(root);
|
||||
const rel = (file: string): string => path.relative(root, file) || path.basename(file);
|
||||
|
||||
const files = await discoverConfigFiles(root);
|
||||
|
||||
+39
-9
@@ -17,7 +17,25 @@ import type {
|
||||
} from "./schema.js";
|
||||
import { toolMap } from "../core/tools.js";
|
||||
|
||||
export const CONFIG_DIRNAME = ".expo-code-review";
|
||||
/**
|
||||
* The setup directory. `.expo-agents/code-review/` is the home from 0.15.0 —
|
||||
* one `.expo-agents/` directory holds every Expo agent tool's per-repo files
|
||||
* (code-review, verify) instead of one dot directory per tool at the root.
|
||||
* The pre-0.15 `.expo-code-review/` keeps working unchanged: every lookup
|
||||
* takes whichever of the two exists (new first), and scaffolding writes into
|
||||
* an existing legacy directory rather than starting a second one.
|
||||
*/
|
||||
export const CONFIG_DIRNAME = path.join(".expo-agents", "code-review");
|
||||
export const LEGACY_CONFIG_DIRNAME = ".expo-code-review";
|
||||
export const CONFIG_DIRNAMES: readonly string[] = [CONFIG_DIRNAME, LEGACY_CONFIG_DIRNAME];
|
||||
|
||||
/** Does this directory path name a setup dir (either spelling)? */
|
||||
export function isConfigDirPath(dir: string): boolean {
|
||||
const normalized = path.normalize(dir);
|
||||
return CONFIG_DIRNAMES.some(
|
||||
(name) => normalized === name || normalized.endsWith(path.sep + name),
|
||||
);
|
||||
}
|
||||
|
||||
/** Stack config for a scope load (where `stack` is schema-rejected and absent). */
|
||||
const STACK_CONFIG_DEFAULTS: LoadedConfig["stack"] = {
|
||||
@@ -56,22 +74,34 @@ const RESEARCH_CONFIG_DEFAULTS: LoadedConfig["research"] = {
|
||||
/** Default OpenCode tool toggles for a reviewer: read the repo, never mutate it. */
|
||||
const DEFAULT_AGENT_TOOLS = toolMap(["read", "grep", "glob", "list"]);
|
||||
|
||||
export function configDirFor(repoRoot: string): string {
|
||||
return path.join(repoRoot, CONFIG_DIRNAME);
|
||||
/**
|
||||
* The setup dir under `base`: `.expo-agents/code-review/` when it exists, else
|
||||
* the legacy `.expo-code-review/` when THAT exists, else the new name (for
|
||||
* messages and for scaffolding a fresh repo). Works for the repo root and for
|
||||
* scope directories alike.
|
||||
*/
|
||||
export function configDirFor(base: string): string {
|
||||
for (const name of CONFIG_DIRNAMES) {
|
||||
const candidate = path.join(base, name);
|
||||
if (existsSync(candidate)) {
|
||||
return candidate;
|
||||
}
|
||||
}
|
||||
return path.join(base, CONFIG_DIRNAME);
|
||||
}
|
||||
|
||||
/**
|
||||
* Resolve the config directory: an explicit override (absolute or repo-relative)
|
||||
* → the ECR_CONFIG_DIR env var → the repo's default `.expo-code-review/`. This is
|
||||
* the escape hatch (graft 1); with neither override present the result is exactly
|
||||
* `configDirFor(repoRoot)`, so the default behavior is byte-identical.
|
||||
* → the ECR_CONFIG_DIR env var → the repo's default setup dir (new name, then
|
||||
* legacy). This is the escape hatch (graft 1); with neither override present the
|
||||
* result is exactly `configDirFor(repoRoot)`.
|
||||
*/
|
||||
export function resolveConfigDir(repoRoot: string, override?: string): string {
|
||||
const chosen = override ?? process.env.ECR_CONFIG_DIR;
|
||||
if (chosen) {
|
||||
return path.isAbsolute(chosen) ? chosen : path.join(repoRoot, chosen);
|
||||
}
|
||||
return path.join(repoRoot, CONFIG_DIRNAME);
|
||||
return configDirFor(repoRoot);
|
||||
}
|
||||
|
||||
export interface LoadConfigOptions {
|
||||
@@ -320,7 +350,7 @@ export function hasScopeConfig(root: string, scope: RoutingScope): boolean {
|
||||
if (scope.config === ".") {
|
||||
return true;
|
||||
}
|
||||
const dir = path.join(root, scope.config, CONFIG_DIRNAME);
|
||||
const dir = configDirFor(path.join(root, scope.config));
|
||||
return existsSync(path.join(dir, "config.jsonc")) || existsSync(path.join(dir, "config.json"));
|
||||
}
|
||||
|
||||
@@ -354,7 +384,7 @@ export async function loadScopeConfig(
|
||||
`scope "${scope.name}": config "${scope.config}" resolves outside the repo checkout`,
|
||||
);
|
||||
}
|
||||
const dir = path.join(root, scope.config, CONFIG_DIRNAME);
|
||||
const dir = configDirFor(path.join(root, scope.config));
|
||||
const { config } = await loadConfigDir(dir, ScopeReviewConfigSchema);
|
||||
base = config;
|
||||
// Non-default scopes never carry their own marker (the scope schema rejects
|
||||
|
||||
+18
-4
@@ -4,7 +4,13 @@
|
||||
import { readdir, readFile, stat } from "node:fs/promises";
|
||||
import path from "node:path";
|
||||
|
||||
import { CONFIG_DIRNAME, stripJsonComments, stripTrailingCommas } from "../config/load.js";
|
||||
import {
|
||||
CONFIG_DIRNAME,
|
||||
LEGACY_CONFIG_DIRNAME,
|
||||
configDirFor,
|
||||
stripJsonComments,
|
||||
stripTrailingCommas,
|
||||
} from "../config/load.js";
|
||||
import { ROUTING_FILENAME } from "../config/routing.js";
|
||||
import { git, pathInside } from "./exec.js";
|
||||
import { matchesIgnore } from "./noise.js";
|
||||
@@ -518,10 +524,18 @@ export async function discoverSetupDirs(root: string): Promise<string[]> {
|
||||
continue;
|
||||
}
|
||||
const child = path.join(dir, entry.name);
|
||||
if (entry.name === CONFIG_DIRNAME) {
|
||||
if (entry.name === LEGACY_CONFIG_DIRNAME) {
|
||||
found.push(child);
|
||||
continue; // a setup dir never nests another
|
||||
}
|
||||
if (entry.name === ".expo-agents") {
|
||||
// The shared Expo agent-tools dir: only its code-review/ child is ours.
|
||||
const setup = path.join(dir, CONFIG_DIRNAME);
|
||||
if ((await pathKind(setup)) === "dir") {
|
||||
found.push(setup);
|
||||
}
|
||||
continue;
|
||||
}
|
||||
// Other dot dirs are skipped wholesale: `.claude/worktrees/` holds checkouts of
|
||||
// this same repo, whose setup dirs would otherwise be swept as if they were scopes.
|
||||
if (entry.name === "node_modules" || entry.name.startsWith(".")) {
|
||||
@@ -617,7 +631,7 @@ async function checkRoutingManifest(
|
||||
}
|
||||
const configDir = typeof entry.config === "string" ? entry.config : null;
|
||||
if (configDir) {
|
||||
const setupDir = path.resolve(index.root, configDir, CONFIG_DIRNAME);
|
||||
const setupDir = configDirFor(path.resolve(index.root, configDir));
|
||||
if (!pathInside(setupDir, index.root)) {
|
||||
problems.push({
|
||||
file: relative,
|
||||
@@ -630,7 +644,7 @@ async function checkRoutingManifest(
|
||||
file: relative,
|
||||
line: 1,
|
||||
kind: "structural",
|
||||
problem: `scope "${name}" has no ${configDir}/${CONFIG_DIRNAME}/ directory`,
|
||||
problem: `scope "${name}" has no ${configDir}/${CONFIG_DIRNAME}/ (or ${LEGACY_CONFIG_DIRNAME}/) directory`,
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
+46
-8
@@ -20,10 +20,10 @@ import type { ResearchMcpRuntime } from "./research.js";
|
||||
export const CLAUDE_CODE_ENGINE = "claude-code" as const;
|
||||
|
||||
/** Fixed so repo config cannot redirect the Meta credential. */
|
||||
// @ref LLP 0003#muse-spark-via-meta-model-api [implements] — fixed endpoint and model allowlist
|
||||
// @ref LLP 0003#muse-spark-via-meta-model-api [implements] — fixed endpoint, provider routing, and known-model metadata
|
||||
export const META_MODEL_API_BASE_URL = "https://api.meta.ai/v1";
|
||||
|
||||
/** Supported public Muse models. Unknown ids remain visible to preflight. */
|
||||
/** Known public Muse models. Other explicit `meta/...` ids use neutral metadata. */
|
||||
export const META_MUSE_MODELS: Record<string, Record<string, unknown>> = {
|
||||
"muse-spark-1.2": {
|
||||
name: "Muse Spark 1.2",
|
||||
@@ -55,6 +55,36 @@ export const META_MUSE_MODELS: Record<string, Record<string, unknown>> = {
|
||||
include: ["reasoning.encrypted_content"],
|
||||
},
|
||||
},
|
||||
"muse-spark-1.3": {
|
||||
name: "Muse Spark 1.3",
|
||||
reasoning: true,
|
||||
limit: { context: 1_048_576, output: 1_048_576 },
|
||||
modalities: {
|
||||
input: ["text", "image", "pdf", "video"],
|
||||
output: ["text"],
|
||||
},
|
||||
options: {
|
||||
store: false,
|
||||
reasoningEffort: "high",
|
||||
reasoningSummary: "auto",
|
||||
include: ["reasoning.encrypted_content"],
|
||||
},
|
||||
},
|
||||
"muse-spark-1.3-contributor": {
|
||||
name: "Muse Spark 1.3 Contributor",
|
||||
reasoning: true,
|
||||
limit: { context: 1_048_576, output: 1_048_576 },
|
||||
modalities: {
|
||||
input: ["text", "image", "pdf", "video"],
|
||||
output: ["text"],
|
||||
},
|
||||
options: {
|
||||
store: false,
|
||||
reasoningEffort: "high",
|
||||
reasoningSummary: "auto",
|
||||
include: ["reasoning.encrypted_content"],
|
||||
},
|
||||
},
|
||||
};
|
||||
|
||||
/**
|
||||
@@ -265,7 +295,19 @@ export function buildOpencodeConfig(
|
||||
entry.tokenEnv &&
|
||||
!entry.upstream
|
||||
) {
|
||||
// Muse needs the Responses adapter; openai-compatible uses Chat Completions.
|
||||
// Meta's Model API uses the Responses adapter; openai-compatible uses Chat
|
||||
// Completions. Register every explicitly referenced `meta/…` model so a new
|
||||
// Meta family/name does not require an ECR release merely to select the right
|
||||
// provider. Known Muse ids get their published capabilities; unknown ids stay
|
||||
// neutral and are validated by Meta on their first request.
|
||||
const models: Record<string, unknown> = {};
|
||||
for (const id of referencedModels) {
|
||||
const slash = id.indexOf("/");
|
||||
if (slash > 0 && id.slice(0, slash) === "meta" && id.length > slash + 1) {
|
||||
const model = id.slice(slash + 1);
|
||||
models[model] = META_MUSE_MODELS[model] ?? {};
|
||||
}
|
||||
}
|
||||
provider.meta = {
|
||||
npm: "@ai-sdk/openai",
|
||||
name: "Meta Model API",
|
||||
@@ -273,11 +315,7 @@ export function buildOpencodeConfig(
|
||||
baseURL: META_MODEL_API_BASE_URL,
|
||||
apiKey: `{env:${entry.tokenEnv}}`,
|
||||
},
|
||||
models: Object.fromEntries(
|
||||
Object.entries(META_MUSE_MODELS).filter(([model]) =>
|
||||
referencedModels.includes(`meta/${model}`),
|
||||
),
|
||||
),
|
||||
models,
|
||||
};
|
||||
continue;
|
||||
}
|
||||
|
||||
@@ -128,6 +128,8 @@ jobs:
|
||||
uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
|
||||
with:
|
||||
name: review-run-log-pr${{ github.event.issue.number }}
|
||||
path: .expo-code-review/.runs/reviews.jsonl
|
||||
path: |
|
||||
.expo-agents/code-review/.runs/reviews.jsonl
|
||||
.expo-code-review/.runs/reviews.jsonl
|
||||
if-no-files-found: ignore
|
||||
retention-days: 14
|
||||
@@ -183,6 +183,8 @@ jobs:
|
||||
uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
|
||||
with:
|
||||
name: review-run-log-pr${{ github.event.issue.number }}
|
||||
path: .expo-code-review/.runs/reviews.jsonl
|
||||
path: |
|
||||
.expo-agents/code-review/.runs/reviews.jsonl
|
||||
.expo-code-review/.runs/reviews.jsonl
|
||||
if-no-files-found: ignore
|
||||
retention-days: 14
|
||||
@@ -87,7 +87,7 @@
|
||||
// "auth": { "mode": "api-key", "provider": "openai", "tokenEnv": "OPENAI_API_KEY" }
|
||||
//
|
||||
// Muse Spark via the public Meta Model API Responses endpoint:
|
||||
// set models to `meta/muse-spark-1.2` and configure:
|
||||
// set models to `meta/muse-spark-1.3` and configure:
|
||||
// "auth": { "providers": {
|
||||
// "meta": { "mode": "api-key", "tokenEnv": "META_API_KEY" }
|
||||
// } }
|
||||
|
||||
@@ -133,6 +133,8 @@ jobs:
|
||||
uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
|
||||
with:
|
||||
name: review-run-log-pr${{ github.event.pull_request.number }}
|
||||
path: .expo-code-review/.runs/reviews.jsonl
|
||||
path: |
|
||||
.expo-agents/code-review/.runs/reviews.jsonl
|
||||
.expo-code-review/.runs/reviews.jsonl
|
||||
if-no-files-found: ignore
|
||||
retention-days: 14
|
||||
Reference in new issue
Block a user