feat(persona): plain npx -y ai-hist-mcp + enable trajectory memory on persona-maker by khaliqgant · Pull Request #211 · AgentWorkforce/workforce

khaliqgant · 2026-06-06T14:28:14Z

What

Now that the ai-hist-mcp wrapper is published with provenance (ai-hist-mcp@0.3.1, SLSA provenance v1, deps ai-hist@0.3.1), simplify the MCP injection and add a real-world memory usage on a built-in persona.

Changes

Injection flip — buildAiHistMcpServer args ['-y','-p','ai-hist','ai-hist-mcp'] → ['-y','ai-hist-mcp'] (type/command/TRAJECTORY_ROOT+AI_HIST_DB env passthrough unchanged). Updated the interactive-spec.test.ts + plan.test.ts assertions. codex-1 verified the published wrapper exposes the full 7-tool contract (search_history, recent_entries, get_session, get_context, stats, search_trajectories, why_for_task) over stdio from a clean npx -y ai-hist-mcp.
persona-maker opts into memory — memory.trajectories: true so it records its authoring decisions (the why). Regenerated workload-router/generated/personas.ts.

Why `aiMemory` is omitted on persona-maker

persona-maker runs on the opencode harness, which doesn't support runtime MCP injection — so aiMemory (the ai-hist MCP recall) would warn-and-skip there. Enabling only trajectories (runtime write-side, harness-agnostic) avoids shipping dead config. For the full why + how combo see examples/review-agent (codex) and examples/proactive-issue-resolver (claude).

Tests

persona-kit 268, workload-router 13 (+ build). persona-maker parse-verified. No runtime change (B's 83/83 unaffected).

🤖 Generated with Claude Code

…rajectory memory on persona-maker The `ai-hist-mcp` wrapper is now published with provenance (ai-hist-mcp@0.3.1, SLSA provenance v1, deps ai-hist@0.3.1), so the injection no longer needs the `-p ai-hist` bootstrap form. - persona-kit buildAiHistMcpServer: args ['-y','-p','ai-hist','ai-hist-mcp'] → ['-y','ai-hist-mcp'] (type/command/env passthrough unchanged); updated the interactive-spec + plan test assertions. codex-1 verified the published wrapper exposes the full 7-tool contract over stdio. - personas/persona-maker.json: opt into memory.trajectories so it records its authoring decisions (the WHY). aiMemory is intentionally omitted here: persona-maker runs on the opencode harness, which doesn't support runtime MCP injection, so the ai-hist MCP recall only applies to claude/codex personas (see examples/review-agent + examples/proactive-issue-resolver for the full why+how combo). Regenerated workload-router generated/personas.ts. Tests: persona-kit 268, workload-router 13. No runtime change. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

codeant-ai · 2026-06-06T14:28:17Z

Your free trial PR review limit of 300 PRs has been reached. Please upgrade your plan to continue using CodeAnt AI.

coderabbitai · 2026-06-06T14:28:20Z

Warning

Review limit reached

@khaliqgant, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 19 minutes and 53 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 95c113b7-7d97-4193-bf93-f0a8bbe957c2

📥 Commits

Reviewing files that changed from the base of the PR and between c074de1 and 377be1d.

⛔ Files ignored due to path filters (1)

packages/workload-router/src/generated/personas.ts is excluded by !**/generated/**

📒 Files selected for processing (4)

packages/persona-kit/src/interactive-spec.test.ts
packages/persona-kit/src/interactive-spec.ts
packages/persona-kit/src/plan.test.ts
personas/persona-maker.json

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch feat/aihist-plain-npx-and-persona-memory

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

gemini-code-assist

Code Review

This pull request simplifies the npx arguments for launching ai-hist-mcp by removing the explicit package flag, and adds a memory configuration block to the persona-maker persona and its generated file. The review feedback recommends pinning the ai-hist-mcp package version (e.g., to ai-hist-mcp@0.3.1) in both the implementation and corresponding tests to prevent network latency and ensure build stability.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

gemini-code-assist · 2026-06-06T14:29:29Z

    type: 'stdio',
    command: 'npx',
-    args: ['-y', '-p', 'ai-hist', 'ai-hist-mcp'],
+    args: ['-y', 'ai-hist-mcp'],


Running npx -y ai-hist-mcp without a pinned version or version range means npx will query the npm registry for the latest tag on every execution. This introduces unnecessary network latency during agent startup and risks breaking existing installations if a future version of ai-hist-mcp introduces breaking changes.\n\nConsider pinning the version to ai-hist-mcp@0.3.1 (or a compatible range like ai-hist-mcp@^0.3.1) to ensure reproducibility, stability, and faster startup times.

Suggested change

args: ['-y', 'ai-hist-mcp'],

args: ['-y', 'ai-hist-mcp@0.3.1'],

gemini-code-assist · 2026-06-06T14:29:30Z

    type: 'stdio',
    command: 'npx',
-    args: ['-y', '-p', 'ai-hist', 'ai-hist-mcp'],
+    args: ['-y', 'ai-hist-mcp'],


Update the test assertion to match the pinned version of ai-hist-mcp.

Suggested change

args: ['-y', 'ai-hist-mcp'],

args: ['-y', 'ai-hist-mcp@0.3.1'],

gemini-code-assist · 2026-06-06T14:29:30Z

    type: 'stdio',
    command: 'npx',
-    args: ['-y', '-p', 'ai-hist', 'ai-hist-mcp'],
+    args: ['-y', 'ai-hist-mcp'],


Update the test assertion to match the pinned version of ai-hist-mcp.

Suggested change

args: ['-y', 'ai-hist-mcp'],

args: ['-y', 'ai-hist-mcp@0.3.1'],

agent-relay-code · 2026-06-06T14:35:56Z

Reviewed PR #211 against the current checkout. I did not find a demonstrated defect requiring code edits.

Validated:

ai-hist-mcp exists on npm and exposes the expected ai-hist-mcp bin.
npx -y ai-hist-mcp starts and responds to a minimal MCP initialize request against a temp TRAJECTORY_ROOT / AI_HIST_DB fixture.
Regenerated workload-router personas.
@agentworkforce/persona-kit tests passed.
@agentworkforce/workload-router tests passed.
Workspace build, lint, typecheck, example typecheck, and recursive tests passed when invoked through corepack pnpm.

Note: corepack pnpm run check itself cannot run in this sandbox because the root npm script shells out to pnpm, and only corepack pnpm is available here. I ran the equivalent phases directly after building workspace declarations.

agent-relay-code · 2026-06-06T14:36:06Z

ℹ️ pr-reviewer: review only — no file changes were applied to the PR (nothing to commit after review). The notes below are advisory and were not pushed.

Reviewed PR #211 against the current checkout. I did not find a demonstrated defect requiring code edits.

Validated:

ai-hist-mcp exists on npm and exposes the expected ai-hist-mcp bin.
npx -y ai-hist-mcp starts and responds to a minimal MCP initialize request against a temp TRAJECTORY_ROOT / AI_HIST_DB fixture.
Regenerated workload-router personas.
@agentworkforce/persona-kit tests passed.
@agentworkforce/workload-router tests passed.
Workspace build, lint, typecheck, example typecheck, and recursive tests passed when invoked through corepack pnpm.

Note: corepack pnpm run check itself cannot run in this sandbox because the root npm script shells out to pnpm, and only corepack pnpm is available here. I ran the equivalent phases directly after building workspace declarations.

Remove the memory.trajectories opt-in added in #211. persona-maker runs on opencode (no MCP injection) and recording it isn't wanted on the built-in; the full why+how usage lives in examples/review-agent + proactive-issue-resolver. Regenerated workload-router generated/personas.ts. Co-authored-by: Ricky Schema Cascade <ricky@agent-relay.com> Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

gemini-code-assist Bot reviewed Jun 6, 2026

View reviewed changes

khaliqgant merged commit 78326b6 into main Jun 6, 2026
3 checks passed

khaliqgant deleted the feat/aihist-plain-npx-and-persona-memory branch June 6, 2026 14:30

khaliqgant mentioned this pull request Jun 6, 2026

chore(persona): drop memory from persona-maker #212

Merged

Navigation Menu

Repository navigation

feat(persona): plain npx -y ai-hist-mcp + enable trajectory memory on persona-maker - #211

feat(persona): plain npx -y ai-hist-mcp + enable trajectory memory on persona-maker#211
khaliqgant merged 1 commit into
mainfrom
feat/aihist-plain-npx-and-persona-memory

khaliqgant commented Jun 6, 2026

Uh oh!

codeant-ai Bot commented Jun 6, 2026

Uh oh!

coderabbitai Bot commented Jun 6, 2026

Review limit reached

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

gemini-code-assist Bot Jun 6, 2026

Uh oh!

gemini-code-assist Bot Jun 6, 2026

Uh oh!

gemini-code-assist Bot Jun 6, 2026

Uh oh!

Uh oh!

agent-relay-code Bot commented Jun 6, 2026

Uh oh!

agent-relay-code Bot commented Jun 6, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

	args: ['-y', 'ai-hist-mcp'],
	args: ['-y', 'ai-hist-mcp@0.3.1'],

Conversation

khaliqgant commented Jun 6, 2026

What

Changes

Why aiMemory is omitted on persona-maker

Tests

Uh oh!

codeant-ai Bot commented Jun 6, 2026

Uh oh!

coderabbitai Bot commented Jun 6, 2026

Review limit reached

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist Bot Jun 6, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jun 6, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Jun 6, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

agent-relay-code Bot commented Jun 6, 2026

Uh oh!

agent-relay-code Bot commented Jun 6, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Why `aiMemory` is omitted on persona-maker