Skip to content

Fix forward #286: derive Grok live-gate reach from runtime ownership #287

Closed

Description

@ericlitman

Introduced by: #286

The merged head of #286 (e4537f0) has Unfret finding v1:427364ab, "New Grok home resolver is omitted from feature selection". .claude/skills/verify-open-pstack/scripts/io.ts, where #286 put grokHome(), is not in GROK_REACH, so a change to it cannot name grok:setup. This is the second consecutive verdict on the same exact-path list. The step-back exchange chose to replace the list.

Acceptance criteria

  1. The exact-path list is gone. GROK_REACH, its loop and GROK_TRANSPORT are deleted from .claude/skills/verify-open-pstack/scripts/core.ts.
  2. Every runtime ownership branch permits the Grok features. Each branch of classify() grants both grok:dispatch and grok:setup. The branches are:
    • catalog;
    • shared and the shared recipe;
    • project;
    • recognized skill content;
    • the transcript resolver.
  3. The narrow cases stay narrow. Asset-only, version-only manifest and non-runtime changes gain no Grok feature, and unmapped paths still fail.
  4. Tests cover the new rule. Every previously omitted path and scripts/io.ts can select both Grok features, and so can a new file under an existing runtime prefix. Self-test adds no Grok session.
  5. The docs match. features/grok.md says that recognized runtime code and instructions permit naming the Grok features, and it names the asset and version-only exceptions instead of listing files.

Out of scope: T3_DISPATCH_PATHS has the same mechanism, as an advisory with no observed failure.

Activity

  1. ericlitman commented on Nov 8, 2026

    @ericlitman
    OwnerAuthor

    PR: #288

  2. added a commit that references this issue on Nov 8, 2026
  3. ericlitman commented on Nov 8, 2026

    @ericlitman
    OwnerAuthor

    #287: Fix forward #286

    The issue body is the only kept version.

    1. GROK_REACH, its loop and GROK_TRANSPORT are deleted. Done as written, in fix(verify): derive Grok live-gate reach from runtime ownership (#287) #288.
    2. Every runtime ownership branch grants both Grok features. Done as written. Proved by the 16-path case in core.test.ts.
    3. Assets, version-only and non-runtime changes stay narrow, and unmapped paths still fail. Done as written. The same test proves it.
    4. The tests. Done as written.
    5. features/grok.md states the ownership rule. Done as written. The live gate on b43e759 selected grok:dispatch through the new rule.

    Batch closeout record: https://github.com/mobilyze-llc/docs/blob/main/projects/open-pstack/records/2026-11-08-batch-281/closeout.md (released in v1.21.0: https://github.com/ericlitman/open-pstack/releases/tag/v1.21.0)

  4. added a commit that references this issue on Nov 9, 2026
    b43e759
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions