@knpkv/codecommit 0.5.5 → 0.7.0

package/README.md

@@ -57,15 +57,16 @@ codecommit web [--port 3000] [--hostname 127.0.0.1]
 codecommit pr list [options]
 ```
 
-| Option      | Alias | Description             | Default     |
-| ----------- | ----- | ----------------------- | ----------- |
-| `--profile` | `-p`  | AWS profile             | `default`   |
-| `--region`  | `-r`  | AWS region              | `us-east-1` |
-| `--status`  | `-s`  | PR status (OPEN/CLOSED) | `OPEN`      |
-| `--all`     | `-a`  | Show all PRs            | `false`     |
-| `--repo`    |       | Filter by repository    | -           |
-| `--author`  |       | Filter by author        | -           |
-| `--json`    |       | Output as JSON          | `false`     |
+| Option      | Alias | Description                                    | Default     |
+| ----------- | ----- | ---------------------------------------------- | ----------- |
+| `--profile` | `-p`  | AWS profile (ignored with --filter)            | `default`   |
+| `--region`  | `-r`  | AWS region (ignored with --filter)             | `us-east-1` |
+| `--status`  | `-s`  | PR status, OPEN/CLOSED (ignored with --filter) | `OPEN`      |
+| `--all`     | `-a`  | Show all PRs (ignored with --filter)           | `false`     |
+| `--repo`    |       | Filter by repository                           | -           |
+| `--author`  |       | Filter by author                               | -           |
+| `--filter`  |       | Named preset, OPEN-only (see below)            | -           |
+| `--json`    |       | Output as JSON                                 | `false`     |
 
 ```bash
 codecommit pr list
@@ -76,6 +77,31 @@ codecommit pr list --author jane
 codecommit pr list --json
 ```
 
+#### Filter presets (`--filter`)
+
+When `--filter` is set, the command fans out across **every enabled account**
+in `~/.codecommit/config.json` (set up via `codecommit tui`) and returns the
+merged list, sorted by last-modified-date. Presets operate on **OPEN PRs only**,
+so `--profile`, `--region`, `--status`, and `--all` are all ignored when
+`--filter` is set. Combine with `--json`, `--repo`, or `--author` for further
+narrowing. If any account fails (e.g. an expired SSO session), a
+`⚠ N account(s) failed` summary is printed to stderr and the PRs from the
+accounts that succeeded are still returned.
+
+| Preset            | Matches                                                                  |
+| ----------------- | ------------------------------------------------------------------------ |
+| `mine`            | Open PRs you authored (matched against `getCallerIdentity` per profile)  |
+| `needs-my-review` | Open PRs awaiting your approval (you're in an unsatisfied approval pool) |
+| `stale`           | Open PRs with no activity for more than 7 days                           |
+| `conflicting`     | Open PRs with merge conflicts                                            |
+
+```bash
+codecommit pr list --filter mine --json          # all my open PRs everywhere
+codecommit pr list --filter needs-my-review      # what I need to review
+codecommit pr list --filter stale --repo my-repo # stale PRs in one repo
+codecommit pr list --filter conflicting --json
+```
+
 Output:
 
 ```

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@knpkv/codecommit",
-    "version": "0.5.5",
+    "version": "0.7.0",
     "description": "TUI for browsing AWS CodeCommit PRs",
     "type": "module",
     "main": "dist/index.js",
@@ -12,7 +12,8 @@
         "access": "public"
     },
     "files": [
-        "dist"
+        "dist",
+        "skills"
     ],
     "scripts": {
         "start": "bun src/bin.ts",
@@ -25,26 +26,25 @@
         "docgen": "docgen"
     },
     "dependencies": {
-        "@effect-atom/atom-react": "latest",
-        "@effect/cli": "latest",
-        "@effect/platform": "latest",
-        "@effect/platform-bun": "latest",
-        "@effect/platform-node": "latest",
+        "@effect/atom-react": "4.0.0-beta.87",
+        "@effect/platform-bun": "4.0.0-beta.87",
+        "@effect/platform-node": "4.0.0-beta.87",
+        "@knpkv/agent-skills": "workspace:^",
         "@knpkv/codecommit-core": "workspace:^",
         "@knpkv/codecommit-web": "workspace:^",
         "@opentui/core": "https://pkg.pr.new/anomalyco/opentui/@opentui/core@367a94087821b3b5feedd35bbb57df43b10a286e",
         "@opentui/react": "https://pkg.pr.new/anomalyco/opentui/@opentui/react@367a94087821b3b5feedd35bbb57df43b10a286e",
-        "effect": "latest",
-        "open": "^10.1.0",
-        "react": "^19.1.0",
+        "effect": "4.0.0-beta.87",
+        "react": "^19.2.7",
         "tslib": "^2.8.1"
     },
     "devDependencies": {
-        "@effect/vitest": "latest",
-        "@storybook/react": "^8.6.15",
-        "@types/bun": "^1.3.6",
+        "@effect/vitest": "4.0.0-beta.87",
+        "@storybook/react": "^10.4.6",
+        "@types/bun": "^1.3.14",
         "@types/node": "latest",
-        "@types/react": "^19.1.8",
-        "vitest": "latest"
+        "@types/react": "^19.2.17",
+        "storybook": "^10.4.6",
+        "vitest": "^4.1.9"
     }
 }

package/dist/src/FilterService.d.ts

@@ -0,0 +1,84 @@
+/**
+ * Cross-account `--filter` orchestration for `pr list`.
+ *
+ * Extracts the fan-out workflow that backs the named filter presets out of the
+ * CLI entrypoint (`bin.ts`) so the entrypoint keeps only presentation. The
+ * service depends on {@link AwsClient.AwsClient} and
+ * {@link ConfigService.ConfigService} and reuses the pure `matchesPreset` /
+ * `matchesRepoAuthor` predicates from `./filterPresets.ts`.
+ *
+ * Behaviour mirrors the original inline block exactly:
+ *
+ * - {@link FilterService.Service.resolveTargets} loads config, keeps only
+ *   enabled accounts, and flattens to `{ profile, region }` targets.
+ * - {@link FilterService.Service.collect} resolves caller identity once per
+ *   profile (for the identity-comparing presets), fans out OPEN-only PR fetches
+ *   across targets (concurrency 4), filters by preset + repo/author, collects
+ *   per-account failures, and returns PRs sorted by `lastModifiedDate` desc.
+ *
+ * @category Service
+ * @module
+ */
+import { AwsClient, ConfigService, type Domain, type Errors } from "@knpkv/codecommit-core";
+import type { AwsProfileName, AwsRegion } from "@knpkv/codecommit-core/Domain.js";
+import { Context, Effect, Layer, type Option } from "effect";
+import { type FilterPreset } from "./filterPresets.js";
+/** A single `{ profile, region }` account/region pair to scan. */
+export interface FilterTarget {
+    readonly profile: AwsProfileName;
+    readonly region: AwsRegion;
+}
+/** Repo/author narrowing options applied on top of the preset. */
+export interface FilterOptions {
+    readonly repo: Option.Option<string>;
+    readonly author: Option.Option<string>;
+}
+/**
+ * Structured outcome of a cross-account scan.
+ *
+ * - `prs`               — matching PRs, sorted by `lastModifiedDate` desc.
+ * - `failures`          — `"<profile>/<region>: <message>"` per failed account.
+ * - `unresolvedProfiles` — profiles whose caller identity didn't resolve, so
+ *   identity-comparing presets may have incomplete results for them.
+ */
+export interface FilterResult {
+    readonly prs: ReadonlyArray<Domain.PullRequest>;
+    readonly failures: ReadonlyArray<string>;
+    readonly unresolvedProfiles: ReadonlyArray<string>;
+}
+type FilterCollectError = Errors.AwsApiError | Errors.AwsCredentialError | Errors.AwsThrottleError;
+/**
+ * Cross-account filter orchestration service.
+ *
+ * @category models
+ */
+export interface FilterServiceShape {
+    readonly resolveTargets: Effect.Effect<ReadonlyArray<FilterTarget>, unknown>;
+    readonly collect: (preset: FilterPreset, targets: ReadonlyArray<FilterTarget>, opts: FilterOptions, now?: Date) => Effect.Effect<FilterResult, FilterCollectError>;
+}
+/**
+ * Cross-account filter orchestration service.
+ *
+ * @category models
+ */
+export declare namespace FilterService {
+    interface Service extends FilterServiceShape {
+    }
+}
+declare const FilterService_base: Context.ServiceClass<FilterService, "@knpkv/codecommit/FilterService", FilterServiceShape>;
+/**
+ * Cross-account filter orchestration service.
+ *
+ * @category Service
+ */
+export declare class FilterService extends FilterService_base {
+}
+/**
+ * Live layer. Requires {@link AwsClient.AwsClient} and
+ * {@link ConfigService.ConfigService} (wired by the caller).
+ *
+ * @category Layer
+ */
+export declare const FilterServiceLive: Layer.Layer<FilterService, Errors.AwsCredentialError | Errors.AwsThrottleError | Errors.AwsApiError, AwsClient.AwsClient | ConfigService.ConfigService>;
+export {};
+//# sourceMappingURL=FilterService.d.ts.map

package/dist/src/FilterService.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"FilterService.d.ts","sourceRoot":"","sources":["../../src/FilterService.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,OAAO,EAAE,SAAS,EAAE,aAAa,EAAE,KAAK,MAAM,EAAE,KAAK,MAAM,EAAE,MAAM,wBAAwB,CAAA;AAC3F,OAAO,KAAK,EAAE,cAAc,EAAE,SAAS,EAAE,MAAM,kCAAkC,CAAA;AACjF,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,KAAK,MAAM,EAAU,MAAM,QAAQ,CAAA;AACpE,OAAO,EAAE,KAAK,YAAY,EAAoC,MAAM,oBAAoB,CAAA;AAExF,kEAAkE;AAClE,MAAM,WAAW,YAAY;IAC3B,QAAQ,CAAC,OAAO,EAAE,cAAc,CAAA;IAChC,QAAQ,CAAC,MAAM,EAAE,SAAS,CAAA;CAC3B;AAED,kEAAkE;AAClE,MAAM,WAAW,aAAa;IAC5B,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAA;IACpC,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAA;CACvC;AAED;;;;;;;GAOG;AACH,MAAM,WAAW,YAAY;IAC3B,QAAQ,CAAC,GAAG,EAAE,aAAa,CAAC,MAAM,CAAC,WAAW,CAAC,CAAA;IAC/C,QAAQ,CAAC,QAAQ,EAAE,aAAa,CAAC,MAAM,CAAC,CAAA;IACxC,QAAQ,CAAC,kBAAkB,EAAE,aAAa,CAAC,MAAM,CAAC,CAAA;CACnD;AAED,KAAK,kBAAkB,GAAG,MAAM,CAAC,WAAW,GAAG,MAAM,CAAC,kBAAkB,GAAG,MAAM,CAAC,gBAAgB,CAAA;AAElG;;;;GAIG;AACH,MAAM,WAAW,kBAAkB;IACjC,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,YAAY,CAAC,EAAE,OAAO,CAAC,CAAA;IAC5E,QAAQ,CAAC,OAAO,EAAE,CAChB,MAAM,EAAE,YAAY,EACpB,OAAO,EAAE,aAAa,CAAC,YAAY,CAAC,EACpC,IAAI,EAAE,aAAa,EACnB,GAAG,CAAC,EAAE,IAAI,KACP,MAAM,CAAC,MAAM,CAAC,YAAY,EAAE,kBAAkB,CAAC,CAAA;CACrD;AAsFD;;;;GAIG;AACH,MAAM,CAAC,OAAO,WAAW,aAAa,CAAC;IACrC,UAAiB,OAAQ,SAAQ,kBAAkB;KAAG;CACvD;;AAED;;;;GAIG;AACH,qBAAa,aAAc,SAAQ,kBAGG;CAAG;AAEzC;;;;;GAKG;AACH,eAAO,MAAM,iBAAiB,yJAAoC,CAAA"}

package/dist/src/FilterService.js

@@ -0,0 +1,85 @@
+/**
+ * Cross-account `--filter` orchestration for `pr list`.
+ *
+ * Extracts the fan-out workflow that backs the named filter presets out of the
+ * CLI entrypoint (`bin.ts`) so the entrypoint keeps only presentation. The
+ * service depends on {@link AwsClient.AwsClient} and
+ * {@link ConfigService.ConfigService} and reuses the pure `matchesPreset` /
+ * `matchesRepoAuthor` predicates from `./filterPresets.ts`.
+ *
+ * Behaviour mirrors the original inline block exactly:
+ *
+ * - {@link FilterService.Service.resolveTargets} loads config, keeps only
+ *   enabled accounts, and flattens to `{ profile, region }` targets.
+ * - {@link FilterService.Service.collect} resolves caller identity once per
+ *   profile (for the identity-comparing presets), fans out OPEN-only PR fetches
+ *   across targets (concurrency 4), filters by preset + repo/author, collects
+ *   per-account failures, and returns PRs sorted by `lastModifiedDate` desc.
+ *
+ * @category Service
+ * @module
+ */
+import { AwsClient, ConfigService } from "@knpkv/codecommit-core";
+import { Context, Effect, Layer, Stream } from "effect";
+import { matchesPreset, matchesRepoAuthor } from "./filterPresets.js";
+const make = Effect.gen(function* () {
+    const aws = yield* AwsClient.AwsClient;
+    const cs = yield* ConfigService.ConfigService;
+    const resolveTargets = Effect.gen(function* () {
+        const config = yield* cs.load;
+        return config.accounts
+            .filter((a) => a.enabled)
+            .flatMap((a) => a.regions.map((r) => ({ profile: a.profile, region: r })));
+    });
+    const collect = (preset, targets, opts, now = new Date()) => Effect.gen(function* () {
+        // Resolve caller identity once per profile (deduped per profile within this
+        // run, not cached across runs) for presets that compare against "me".
+        const callerByProfile = new Map();
+        // Profiles whose caller-identity didn't resolve (lookup failed or returned
+        // no username). For the identity-comparing presets this means their PRs
+        // can't be matched, so we surface a warning rather than silently dropping them.
+        const unresolvedCallerProfiles = [];
+        if (preset === "mine" || preset === "needs-my-review") {
+            const uniqueProfiles = [...new Map(targets.map((t) => [t.profile, t])).values()];
+            const callers = yield* Effect.forEach(uniqueProfiles, (acct) => Effect.catchIf(Effect.map(aws.getCallerIdentity(acct), (id) => ({
+                profile: acct.profile,
+                username: id.username
+            })), () => true, () => Effect.succeed({ profile: acct.profile, username: null }), () => Effect.succeed({ profile: acct.profile, username: null })), { concurrency: 4 });
+            for (const { profile: p, username } of callers) {
+                if (username)
+                    callerByProfile.set(p, username);
+                else
+                    unresolvedCallerProfiles.push(p);
+            }
+        }
+        const collected = yield* Effect.forEach(targets, (acct) => Effect.catchIf(aws.getPullRequests(acct, { status: "OPEN" }).pipe(Stream.filter((pr) => matchesPreset(preset, pr, callerByProfile, now) && matchesRepoAuthor(pr, opts.repo, opts.author)), Stream.runCollect, Effect.map((chunk) => ({ ok: Array.from(chunk), failed: null }))), () => true, 
+        // Don't silently coalesce auth/permission failures to "no matches" —
+        // collect the failure so it can be surfaced after the results.
+        (e) => Effect.succeed({
+            ok: [],
+            failed: `${acct.profile}/${acct.region}: ${e.message}`
+        }), (e) => Effect.succeed({
+            ok: [],
+            failed: `${acct.profile}/${acct.region}: ${e.message}`
+        })), { concurrency: 4 });
+        const prs = collected.flatMap((r) => r.ok).sort((a, b) => b.lastModifiedDate.getTime() - a.lastModifiedDate.getTime());
+        const failures = collected.flatMap((r) => (r.failed === null ? [] : [r.failed]));
+        return { prs, failures, unresolvedProfiles: unresolvedCallerProfiles };
+    });
+    return { resolveTargets, collect };
+});
+/**
+ * Cross-account filter orchestration service.
+ *
+ * @category Service
+ */
+export class FilterService extends Context.Service()("@knpkv/codecommit/FilterService") {
+}
+/**
+ * Live layer. Requires {@link AwsClient.AwsClient} and
+ * {@link ConfigService.ConfigService} (wired by the caller).
+ *
+ * @category Layer
+ */
+export const FilterServiceLive = Layer.effect(FilterService, make);
+//# sourceMappingURL=FilterService.js.map

package/dist/src/FilterService.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"FilterService.js","sourceRoot":"","sources":["../../src/FilterService.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,OAAO,EAAE,SAAS,EAAE,aAAa,EAA4B,MAAM,wBAAwB,CAAA;AAE3F,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAe,MAAM,EAAE,MAAM,QAAQ,CAAA;AACpE,OAAO,EAAqB,aAAa,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAA;AA6CxF,MAAM,IAAI,GAIN,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC;IACtB,MAAM,GAAG,GAAG,KAAK,CAAC,CAAC,SAAS,CAAC,SAAS,CAAA;IACtC,MAAM,EAAE,GAAG,KAAK,CAAC,CAAC,aAAa,CAAC,aAAa,CAAA;IAE7C,MAAM,cAAc,GAAG,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC;QACzC,MAAM,MAAM,GAAG,KAAK,CAAC,CAAC,EAAE,CAAC,IAAI,CAAA;QAC7B,OAAO,MAAM,CAAC,QAAQ;aACnB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC;aACxB,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,EAAgB,EAAE,CAAC,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC,OAAO,EAAE,MAAM,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAA;IAC5F,CAAC,CAAC,CAAA;IAEF,MAAM,OAAO,GAAkC,CAAC,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,GAAG,GAAG,IAAI,IAAI,EAAE,EAAE,EAAE,CACzF,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC;QAClB,4EAA4E;QAC5E,sEAAsE;QACtE,MAAM,eAAe,GAAG,IAAI,GAAG,EAAkB,CAAA;QACjD,2EAA2E;QAC3E,wEAAwE;QACxE,gFAAgF;QAChF,MAAM,wBAAwB,GAAkB,EAAE,CAAA;QAClD,IAAI,MAAM,KAAK,MAAM,IAAI,MAAM,KAAK,iBAAiB,EAAE,CAAC;YACtD,MAAM,cAAc,GAAG,CAAC,GAAG,IAAI,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAA;YAChF,MAAM,OAAO,GAAG,KAAK,CAAC,CAAC,MAAM,CAAC,OAAO,CACnC,cAAc,EACd,CAAC,IAAI,EAAE,EAAE,CACP,MAAM,CAAC,OAAO,CACZ,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,iBAAiB,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,EAAgD,EAAE,CAAC,CAAC;gBAC7F,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,QAAQ,EAAE,EAAE,CAAC,QAAQ;aACtB,CAAC,CAAC,EACH,GAAG,EAAE,CAAC,IAAI,EACV,GAAG,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,EAAE,QAAQ,EAAE,IAAqB,EAAE,CAAC,EAChF,GAAG,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,EAAE,QAAQ,EAAE,IAAqB,EAAE,CAAC,CACjF,EACH,EAAE,WAAW,EAAE,CAAC,EAAE,CACnB,CAAA;YACD,KAAK,MAAM,EAAE,OAAO,EAAE,CAAC,EAAE,QAAQ,EAAE,IAAI,OAAO,EAAE,CAAC;gBAC/C,IAAI,QAAQ;oBAAE,eAAe,CAAC,GAAG,CAAC,CAAC,EAAE,QAAQ,CAAC,CAAA;;oBACzC,wBAAwB,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;YACvC,CAAC;QACH,CAAC;QAED,MAAM,SAAS,GAAG,KAAK,CAAC,CAAC,MAAM,CAAC,OAAO,CACrC,OAAO,EACP,CAAC,IAAI,EAAE,EAAE,CACP,MAAM,CAAC,OAAO,CACZ,GAAG,CAAC,eAAe,CAAC,IAAI,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC,CAAC,IAAI,CAChD,MAAM,CAAC,MAAM,CAAC,CAAC,EAAE,EAAE,EAAE,CACnB,aAAa,CAAC,MAAM,EAAE,EAAE,EAAE,eAAe,EAAE,GAAG,CAAC,IAAI,iBAAiB,CAAC,EAAE,EAAE,IAAI,CAAC,IAAI,EAAE,IAAI,CAAC,MAAM,CAAC,CACjG,EACD,MAAM,CAAC,UAAU,EACjB,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,EAAE,EAAE,EAAE,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,EAAE,MAAM,EAAE,IAAqB,EAAE,CAAC,CAAC,CAClF,EACD,GAAG,EAAE,CAAC,IAAI;QACV,qEAAqE;QACrE,+DAA+D;QAC/D,CAAC,CAAC,EAAE,EAAE,CACJ,MAAM,CAAC,OAAO,CAAC;YACb,EAAE,EAAE,EAA+B;YACnC,MAAM,EAAE,GAAG,IAAI,CAAC,OAAO,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,CAAC,OAAO,EAAE;SACvD,CAAC,EACJ,CAAC,CAAC,EAAE,EAAE,CACJ,MAAM,CAAC,OAAO,CAAC;YACb,EAAE,EAAE,EAA+B;YACnC,MAAM,EAAE,GAAG,IAAI,CAAC,OAAO,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,CAAC,OAAO,EAAE;SACvD,CAAC,CACL,EACH,EAAE,WAAW,EAAE,CAAC,EAAE,CACnB,CAAA;QACD,MAAM,GAAG,GAAG,SAAS,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CACvD,CAAC,CAAC,gBAAgB,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC,gBAAgB,CAAC,OAAO,EAAE,CAC5D,CAAA;QACD,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,MAAM,KAAK,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAA;QAEhF,OAAO,EAAE,GAAG,EAAE,QAAQ,EAAE,kBAAkB,EAAE,wBAAwB,EAAE,CAAA;IACxE,CAAC,CAAoD,CAAA;IAEvD,OAAO,EAAE,cAAc,EAAE,OAAO,EAAwC,CAAA;AAC1E,CAAC,CAAC,CAAA;AAWF;;;;GAIG;AACH,MAAM,OAAO,aAAc,SAAQ,OAAO,CAAC,OAAO,EAG/C,CAAC,iCAAiC,CAAC;CAAG;AAEzC;;;;;GAKG;AACH,MAAM,CAAC,MAAM,iBAAiB,GAAG,KAAK,CAAC,MAAM,CAAC,aAAa,EAAE,IAAI,CAAC,CAAA"}

package/dist/src/bin.js

@@ -1,37 +1,46 @@
 #!/usr/bin/env bun
-import { Args, Command, Options } from "@effect/cli";
-import { FileSystem } from "@effect/platform";
-import { BunContext, BunRuntime } from "@effect/platform-bun";
+import { BunRuntime, BunServices } from "@effect/platform-bun";
 import { NodeHttpClient } from "@effect/platform-node";
-import { AwsClient, AwsClientConfig } from "@knpkv/codecommit-core";
+import { makeInstallCommand } from "@knpkv/agent-skills";
+import { AwsClient, AwsClientConfig, CacheService, ConfigService } from "@knpkv/codecommit-core";
 import { makeServer } from "@knpkv/codecommit-web";
 import { Console, Effect, Layer, Stream } from "effect";
-import open from "open";
+import * as FileSystem from "effect/FileSystem";
+import * as Runtime from "effect/Runtime";
+import * as Stdio from "effect/Stdio";
+import { Argument as Args, Command, Flag as Options } from "effect/unstable/cli";
+import * as ChildProcess from "effect/unstable/process/ChildProcess";
 import pkg from "../package.json";
+import { FILTER_PRESETS, matchesRepoAuthor } from "./filterPresets.js";
+import { FilterService, FilterServiceLive } from "./FilterService.js";
 // TUI Command
-const tui = Command.make("tui", {}, () => Effect.gen(function* () {
-    yield* Effect.promise(() => import("./main.js"));
-}));
+const launchTui = Effect.gen(function* () {
+    const { default: program } = yield* Effect.promise(() => import("./main.js"));
+    yield* program;
+});
+const tui = Command.make("tui", {}, () => launchTui);
 // Web Command
 const web = Command.make("web", {
     port: Options.integer("port").pipe(Options.withDefault(3000)),
-    hostname: Options.text("hostname").pipe(Options.withDefault("127.0.0.1"))
+    hostname: Options.string("hostname").pipe(Options.withDefault("127.0.0.1"))
 }, ({ hostname, port }) => Effect.gen(function* () {
     yield* Effect.logInfo(`Starting web server at http://${hostname}:${port}`);
     // Open browser
-    yield* Effect.promise(() => open(`http://${hostname}:${port}`));
+    const url = `http://${hostname}:${port}`;
+    const exitCode = (command) => Effect.scoped(command.pipe(Effect.flatMap((handle) => handle.exitCode)));
+    yield* exitCode(ChildProcess.make("open", [url])).pipe(Effect.catchIf(() => true, () => exitCode(ChildProcess.make("xdg-open", [url]))), Effect.catchIf(() => true, () => exitCode(ChildProcess.make("rundll32.exe", ["url.dll,FileProtocolHandler", url]))), Effect.catchIf(() => true, () => Effect.void));
     // Run server with configured port/hostname
     return yield* Layer.launch(makeServer({ port, hostname }));
 }));
 // PR Create Command
 const prCreate = Command.make("create", {
-    repo: Args.text({ name: "repository" }).pipe(Args.withDescription("Repository name")),
-    title: Args.text({ name: "title" }).pipe(Args.withDescription("PR title")),
-    source: Options.text("source").pipe(Options.withAlias("s"), Options.withDescription("Source branch")),
-    destination: Options.text("destination").pipe(Options.withAlias("d"), Options.withDescription("Destination branch"), Options.withDefault("main")),
-    description: Options.text("description").pipe(Options.withDescription("PR description"), Options.optional),
-    profile: Options.text("profile").pipe(Options.withAlias("p"), Options.withDescription("AWS profile"), Options.withDefault("default")),
-    region: Options.text("region").pipe(Options.withAlias("r"), Options.withDescription("AWS region"), Options.withDefault("us-east-1"))
+    repo: Args.string("repository").pipe(Args.withDescription("Repository name")),
+    title: Args.string("title").pipe(Args.withDescription("PR title")),
+    source: Options.string("source").pipe(Options.withAlias("s"), Options.withDescription("Source branch")),
+    destination: Options.string("destination").pipe(Options.withAlias("d"), Options.withDescription("Destination branch"), Options.withDefault("main")),
+    description: Options.string("description").pipe(Options.withDescription("PR description"), Options.optional),
+    profile: Options.string("profile").pipe(Options.withAlias("p"), Options.withDescription("AWS profile"), Options.withDefault("default")),
+    region: Options.string("region").pipe(Options.withAlias("r"), Options.withDescription("AWS region"), Options.withDefault("us-east-1"))
 }, ({ description, destination, profile, region, repo, source, title }) => Effect.gen(function* () {
     const aws = yield* AwsClient.AwsClient;
     yield* Console.log(`Creating PR: ${source} -> ${destination} in ${repo}`);
@@ -46,28 +55,83 @@ const prCreate = Command.make("create", {
     const link = `https://${region}.console.aws.amazon.com/codesuite/codecommit/repositories/${repo}/pull-requests/${prId}?region=${region}`;
     yield* Console.log(`Created PR: ${prId}`);
     yield* Console.log(link);
-}).pipe(Effect.provide(Layer.merge(AwsClient.AwsClientLive, NodeHttpClient.layer)))).pipe(Command.withDescription("Create a pull request"));
+}).pipe(Effect.provide(Layer.merge(AwsClient.AwsClientLive, NodeHttpClient.layerFetch)))).pipe(Command.withDescription("Create a pull request"));
+// Filter presets (FILTER_PRESETS, matchesPreset, matchesRepoAuthor) live in
+// ./filterPresets.ts — a side-effect-free module so they can be unit-tested
+// without importing this CLI entrypoint (which boots the TUI/Bun runtime).
 // PR List Command
 const prList = Command.make("list", {
-    profile: Options.text("profile").pipe(Options.withAlias("p"), Options.withDescription("AWS profile"), Options.withDefault("default")),
-    region: Options.text("region").pipe(Options.withAlias("r"), Options.withDescription("AWS region"), Options.withDefault("us-east-1")),
-    status: Options.choice("status", ["OPEN", "CLOSED"]).pipe(Options.withAlias("s"), Options.withDescription("Filter by PR status"), Options.withDefault("OPEN")),
-    all: Options.boolean("all").pipe(Options.withAlias("a"), Options.withDescription("Show all PRs (both OPEN and CLOSED)"), Options.withDefault(false)),
-    repo: Options.text("repo").pipe(Options.withDescription("Filter by repository name"), Options.optional),
-    author: Options.text("author").pipe(Options.withDescription("Filter by author"), Options.optional),
+    profile: Options.string("profile").pipe(Options.withAlias("p"), Options.withDescription("AWS profile (ignored when --filter is set — presets fan out across all enabled accounts)"), Options.withDefault("default")),
+    region: Options.string("region").pipe(Options.withAlias("r"), Options.withDescription("AWS region (ignored when --filter is set — presets fan out across all enabled accounts)"), Options.withDefault("us-east-1")),
+    status: Options.choice("status", ["OPEN", "CLOSED"]).pipe(Options.withAlias("s"), Options.withDescription("Filter by PR status (ignored when --filter is set — presets are OPEN-only)"), Options.withDefault("OPEN")),
+    all: Options.boolean("all").pipe(Options.withAlias("a"), Options.withDescription("Show all PRs (both OPEN and CLOSED; ignored when --filter is set — presets are OPEN-only)"), Options.withDefault(false)),
+    repo: Options.string("repo").pipe(Options.withDescription("Filter by repository name"), Options.optional),
+    author: Options.string("author").pipe(Options.withDescription("Filter by author"), Options.optional),
+    filter: Options.choice("filter", FILTER_PRESETS).pipe(Options.withDescription("Named preset (fans out across all enabled accounts, OPEN PRs only — ignores --status/--all): " +
+        "mine | needs-my-review | stale | conflicting"), Options.optional),
     json: Options.boolean("json").pipe(Options.withDescription("Output as JSON"), Options.withDefault(false))
-}, ({ all, author, json, profile, region, repo, status }) => Effect.gen(function* () {
+}, ({ all, author, filter, json, profile, region, repo, status }) => Effect.gen(function* () {
     const aws = yield* AwsClient.AwsClient;
+    // ── Filter-preset path: fan out across all enabled accounts ──────────────
+    if (filter._tag === "Some") {
+        const preset = filter.value;
+        const fs = yield* FilterService;
+        const targets = yield* fs.resolveTargets;
+        if (targets.length === 0) {
+            yield* Console.log("No enabled accounts in ~/.codecommit/config.json. Enable some with `codecommit tui`.");
+            return;
+        }
+        // Progress/status text goes to stderr so `--json` emits only the JSON document on stdout.
+        if (!json)
+            yield* Console.error(`Scanning ${targets.length} account(s) with filter '${preset}'...`);
+        const { failures, prs, unresolvedProfiles } = yield* fs.collect(preset, targets, { repo, author });
+        const unresolvedCallerProfiles = unresolvedProfiles;
+        // Warn (on stderr, so `--json` stdout stays clean) when caller identity
+        // couldn't be resolved for an identity-comparing preset — those accounts'
+        // results may be incomplete because no PR can match an unknown "me".
+        const reportWarnings = Effect.gen(function* () {
+            for (const p of unresolvedCallerProfiles) {
+                yield* Console.error(`⚠ could not resolve caller identity for profile ${p}; '${preset}' results for it may be incomplete`);
+            }
+            if (failures.length > 0) {
+                yield* Console.error(`\n⚠ ${failures.length} account(s) failed:`);
+                for (const f of failures)
+                    yield* Console.error(`  ${f}`);
+            }
+        });
+        if (prs.length === 0) {
+            if (json)
+                yield* Console.log("[]");
+            else
+                yield* Console.error(`No PRs match filter '${preset}'.`);
+            yield* reportWarnings;
+            return;
+        }
+        if (json) {
+            yield* Console.log(JSON.stringify(prs, null, 2));
+        }
+        else {
+            yield* Console.log(`\nFound ${prs.length} PR(s) matching filter '${preset}':\n`);
+            for (const pr of prs) {
+                const flags = [
+                    pr.isApproved ? "approved" : "",
+                    pr.isMergeable ? "mergeable" : "conflicts"
+                ].filter(Boolean).join(" ");
+                yield* Console.log(`${pr.id}  ${pr.repositoryName}  [${pr.account.profile}/${pr.account.region}]`);
+                yield* Console.log(`    ${pr.title}`);
+                yield* Console.log(`    ${pr.sourceBranch} -> ${pr.destinationBranch}`);
+                yield* Console.log(`    by ${pr.author}  ${flags}`);
+                yield* Console.log("");
+            }
+        }
+        yield* reportWarnings;
+        return;
+    }
+    // ── Single-account path (original behaviour) ─────────────────────────────
     const account = { profile: profile, region: region };
     const statusLabel = all ? "all" : status.toLowerCase();
     yield* Console.log(`Fetching ${statusLabel} PRs...`);
-    const filterPRs = (prStream) => prStream.pipe(Stream.filter((pr) => {
-        if (repo._tag === "Some" && pr.repositoryName !== repo.value)
-            return false;
-        if (author._tag === "Some" && pr.author !== author.value)
-            return false;
-        return true;
-    }), Stream.runCollect, Effect.map((chunk) => Array.from(chunk)));
+    const filterPRs = (prStream) => prStream.pipe(Stream.filter((pr) => matchesRepoAuthor(pr, repo, author)), Stream.runCollect, Effect.map((chunk) => Array.from(chunk)));
     let prs;
     if (all) {
         const [openPrs, closedPrs] = yield* Effect.all([
@@ -101,7 +165,10 @@ const prList = Command.make("list", {
             yield* Console.log("");
         }
     }
-}).pipe(Effect.provide(Layer.merge(AwsClient.AwsClientLive, NodeHttpClient.layer)))).pipe(Command.withDescription("List open pull requests"));
+}).pipe(Effect.provide(
+// FilterService draws AwsClient/ConfigService from the base layers, which
+// are also merged into the output so the single-account path keeps them.
+FilterServiceLive.pipe(Layer.provideMerge(Layer.mergeAll(AwsClient.AwsClientLive, NodeHttpClient.layerFetch, ConfigService.ConfigServiceLive.pipe(Layer.provide(CacheService.EventsHub.Default)))))))).pipe(Command.withDescription("List pull requests (use --filter for cross-account presets)"));
 // Helper to render comment threads as markdown
 const renderThread = (thread, indent = 0) => {
     const prefix = "  ".repeat(indent);
@@ -120,11 +187,11 @@ const renderLocation = (loc) => {
 };
 // PR Export Command
 const prExport = Command.make("export", {
-    prId: Args.text({ name: "pr-id" }).pipe(Args.withDescription("Pull request ID")),
-    repo: Args.text({ name: "repository" }).pipe(Args.withDescription("Repository name")),
+    prId: Args.string("pr-id").pipe(Args.withDescription("Pull request ID")),
+    repo: Args.string("repository").pipe(Args.withDescription("Repository name")),
     output: Options.file("output").pipe(Options.withAlias("o"), Options.withDescription("Output file path"), Options.optional),
-    profile: Options.text("profile").pipe(Options.withAlias("p"), Options.withDescription("AWS profile"), Options.withDefault("default")),
-    region: Options.text("region").pipe(Options.withAlias("r"), Options.withDescription("AWS region"), Options.withDefault("us-east-1"))
+    profile: Options.string("profile").pipe(Options.withAlias("p"), Options.withDescription("AWS profile"), Options.withDefault("default")),
+    region: Options.string("region").pipe(Options.withAlias("r"), Options.withDescription("AWS region"), Options.withDefault("us-east-1"))
 }, ({ output, prId, profile, region, repo }) => Effect.gen(function* () {
     const aws = yield* AwsClient.AwsClient;
     const fs = yield* FileSystem.FileSystem;
@@ -166,14 +233,14 @@ const prExport = Command.make("export", {
         yield* Console.log("");
         yield* Console.log(markdown);
     }
-}).pipe(Effect.provide(Layer.merge(AwsClient.AwsClientLive, NodeHttpClient.layer)))).pipe(Command.withDescription("Export PR comments as markdown"));
+}).pipe(Effect.provide(Layer.merge(AwsClient.AwsClientLive, NodeHttpClient.layerFetch)))).pipe(Command.withDescription("Export PR comments as markdown"));
 // PR Update Command
 const prUpdate = Command.make("update", {
-    prId: Args.text({ name: "pr-id" }).pipe(Args.withDescription("Pull request ID")),
-    title: Options.text("title").pipe(Options.withAlias("t"), Options.withDescription("New PR title"), Options.optional),
-    description: Options.text("description").pipe(Options.withAlias("d"), Options.withDescription("New PR description"), Options.optional),
-    profile: Options.text("profile").pipe(Options.withAlias("p"), Options.withDescription("AWS profile"), Options.withDefault("default")),
-    region: Options.text("region").pipe(Options.withAlias("r"), Options.withDescription("AWS region"), Options.withDefault("us-east-1"))
+    prId: Args.string("pr-id").pipe(Args.withDescription("Pull request ID")),
+    title: Options.string("title").pipe(Options.withAlias("t"), Options.withDescription("New PR title"), Options.optional),
+    description: Options.string("description").pipe(Options.withAlias("d"), Options.withDescription("New PR description"), Options.optional),
+    profile: Options.string("profile").pipe(Options.withAlias("p"), Options.withDescription("AWS profile"), Options.withDefault("default")),
+    region: Options.string("region").pipe(Options.withAlias("r"), Options.withDescription("AWS region"), Options.withDefault("us-east-1"))
 }, ({ description, prId, profile, region, title }) => Effect.gen(function* () {
     const aws = yield* AwsClient.AwsClient;
     const account = { profile: profile, region: region };
@@ -190,15 +257,35 @@ const prUpdate = Command.make("update", {
         yield* aws.updatePullRequestDescription({ account, pullRequestId: prId, description: description.value });
     }
     yield* Console.log(`Updated PR ${prId}`);
-}).pipe(Effect.provide(Layer.merge(AwsClient.AwsClientLive, NodeHttpClient.layer)))).pipe(Command.withDescription("Update PR title or description"));
+}).pipe(Effect.provide(Layer.merge(AwsClient.AwsClientLive, NodeHttpClient.layerFetch)))).pipe(Command.withDescription("Update PR title or description"));
 // PR Command (parent)
 const pr = Command.make("pr", {}, () => Console.log("Usage: codecommit pr <command>")).pipe(Command.withSubcommands([prList, prCreate, prExport, prUpdate]), Command.withDescription("Pull request commands"));
+const skillsInstall = makeInstallCommand({
+    description: "Install the CodeCommit agent skill",
+    name: "install",
+    skills: ["codecommit"]
+});
+const skills = Command.make("skills", {}, () => Console.log("Usage: codecommit skills install")).pipe(Command.withSubcommands([skillsInstall]), Command.withDescription("Agent skill commands"));
 const command = Command.make("codecommit", {}, () => 
 // Default to TUI if no subcommand
-Effect.promise(() => import("./main.js"))).pipe(Command.withSubcommands([tui, web, pr]));
-const cli = Command.run(command, {
-    name: pkg.name,
+launchTui).pipe(Command.withSubcommands([tui, web, pr, skills]));
+const cli = Command.runWith(command, {
     version: pkg.version
 });
-Effect.suspend(() => cli(process.argv)).pipe(Effect.provide(Layer.mergeAll(BunContext.layer, NodeHttpClient.layer, AwsClientConfig.Default)), BunRuntime.runMain);
+const AppRuntimeLayer = Layer.mergeAll(NodeHttpClient.layerFetch, AwsClientConfig.Default);
+const needsAppRuntime = (args) => args[0] !== "skills";
+const program = Effect.gen(function* () {
+    const stdio = yield* Stdio.Stdio;
+    const args = yield* stdio.args;
+    const runCli = needsAppRuntime(args) ? cli(args).pipe(Effect.provide(AppRuntimeLayer)) : cli(args);
+    return yield* runCli;
+});
+// The TUI keeps long-lived resources open through its atom runtime (SQLite
+// repos, the HTTP client, the EventsHub PubSub). When the user quits in-app the
+// main fiber exits cleanly (code 0) and — because OpenTUI holds stdin in raw
+// mode, so Ctrl-C is delivered as a keypress, not a SIGINT — runMain's default
+// teardown never reaches `process.exit`. The process would then hang on those
+// open handles after the UI has already torn down. Always terminate explicitly.
+const forceExitTeardown = (exit) => Runtime.defaultTeardown(exit, (code) => process.exit(code));
+BunRuntime.runMain(Effect.provide(program, BunServices.layer), { teardown: forceExitTeardown });
 //# sourceMappingURL=bin.js.map