@agentsfleet/orly 0.5.0 → 0.5.2

package/README.md

@@ -39,9 +39,19 @@ Works with Claude Code, Codex, OpenCode, and Amp.
 | a coding agent | something has to read the rules | Claude Code, Codex, OpenCode, or Amp |
 | your own check commands | `orly gate` runs what `.oracle/orly.json` names | whatever your repository already runs |
 
-gstack is not needed to install or run orly. The rules it installs do route the
-review stage through `gstack /review`. Without gstack, record the skip in your
-Pull Request notes and rerun before merge.
+### gstack
+
+[gstack](https://github.com/garrytan/gstack) is **optional**. orly does not
+require it to install or run.
+
+orly's rules route the review stage through `gstack /review` when gstack is
+available. If it is installed, `/review` is detected and runs automatically.
+If not, orly records the skipped review in the Pull Request notes so it can be
+rerun before merge.
+
+**Bottom line:** orly works without gstack. Installing
+[gstack](https://github.com/garrytan/gstack) gives you the automated `/review`
+step.
 
 ```bash
 bunx @agentsfleet/orly init

package/docs/EXECUTE_DOC_READS.md

@@ -7,7 +7,7 @@ Every triggered document requires one `📖 DOC READ: <path>` proof-line **per t
 | Trigger | Read |
 |---|---|
 | Always (universal) | `docs/greptile-learnings/RULES.md` — at EXECUTE start read the Rule-code gloss legend + the sections named by the spec's "Applicable Rules" list (canonical); the full file belongs to `/review`, not EXECUTE. On sub-task shape change, section-scan (`grep -n "^## "`) and read only newly-touched sections, cited in the proof-line — never the full file again. |
-| `core/**`, `packs/**`, `schemas/**`, `src/**`, `registry.json`, agent-home instruction links, repository rules synchronization, or governance hooks | `dispatch/edit_rules.md` + `docs/ORLY_ARCHITECTURE.md` — canonical sources, profiles, snapshots, refusal boundaries, and evidence. |
+| `core/**`, `packs/**`, `schemas/**`, `src/**`, `registry.json`, agent-home instruction links, repository rules synchronization, or governance hooks | `dispatch/edit_rules.md` + `docs/ORLY_ARCHITECTURE.md` — canonical sources, profiles, snapshots, refusal boundaries, and evidence. | <!-- oracle-packs:workflow.governance -->
 | Lifecycle stage transition (CHORE(open)/PLAN/CHORE(close)/LAND, worktree setup, milestone bootstrap) | `dispatch/lifecycle.md` — the entered stage's runbook section only (section-scan; fires on the transition itself, not on an Edit/Write). |
 | Any source file (cross-cutting authoring) | `dispatch/write_any.md` — length, logging, milestone-id, error-registry, UFS, greptile read, legacy-workaround family. |
 | Spec's "Applicable Rules" | Each rule (canonical). Missing → the cross-cutting codes of `dispatch/write_any.md` are the floor; surface omission. |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentsfleet/orly",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "Repository-scoped engineering harness: renders agent rules, materialises the gates that enforce them, and proves the PR boundary — for any coding agent.",
   "license": "MIT",
   "type": "module",

package/src/install_packs.test.ts

@@ -0,0 +1,35 @@
+import { afterEach, describe, expect, test } from "bun:test";
+import { existsSync } from "node:fs";
+import { join } from "node:path";
+
+import { cleanupTemporaryDirectories, newRepository, ROOT } from "./gates_test_support";
+import { install } from "./install";
+import { RulesModel } from "./model";
+
+const SOURCE_EXTENSIONS = ["rs", "ts", "tsx", "js", "jsx", "py", "sh", "sql", "zig", "mdx"];
+const ENGINE_VERSION = "0.5.2";
+
+afterEach(cleanupTemporaryDirectories);
+
+describe("opt-in pack selection", () => {
+  // An opt-in pack may cite a file another opt-in pack owns, and every such
+  // citation has to be gated by the pack that provides it. Ungated, the install
+  // is refused for skipping rules the repository was never meant to take:
+  // `product.agentsfleet` ships the doc-read table, `workflow.governance` ships
+  // the governance façade that table once named unconditionally.
+  test("one opt-in pack installs without dragging in the others", async () => {
+    const model = await RulesModel.load(ROOT);
+    const repo = newRepository();
+    for (const extension of SOURCE_EXTENSIONS) await Bun.write(join(repo, `src/source.${extension}`), "\n");
+    await Bun.write(join(repo, ".oracle/orly.json"), JSON.stringify({ schema_version: 1, orly_version: "", packs: ["product.agentsfleet"], commands: {}, managed: [] }));
+
+    const result = await install(model, { targetRoot: repo, force: false, installHooks: true, orlyVersion: ENGINE_VERSION });
+
+    expect(result.errors).toEqual([]);
+    expect(result.ok).toBe(true);
+    expect(result.packs).toContain("product.agentsfleet");
+    expect(result.packs).not.toContain("workflow.governance");
+    expect(existsSync(join(repo, "docs/EXECUTE_DOC_READS.md"))).toBe(true);
+    expect(existsSync(join(repo, "dispatch/edit_rules.md"))).toBe(false);
+  });
+});