Skip to content

Commit f21de15

Browse files
committed
fix: fix release
1 parent 0f45620 commit f21de15

3 files changed

Lines changed: 23 additions & 2 deletions

File tree

docs/known-issues.md

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,3 +42,22 @@ Path A is the cheaper restore. Punt the decision until after the Rust port settl
4242
cp -r demos/pagerduty-triage/dist/. website/dist/demos/pagerduty-triage/
4343
```
4444
3. Delete this section from `known-issues.md`.
45+
46+
## `@graphrefly/cli` and `@graphrefly/mcp-server` marked private
47+
48+
**Symptom:** changesets `pnpm release` fails with TypeScript build errors when trying to publish these packages — `Property 'explain' does not exist on type 'Graph'`, `Module '"@graphrefly/graphrefly"' has no exported member 'memoryStorage'`, `Module '"@graphrefly/graphrefly/extra/node"' has no exported member 'fileStorage'`, etc.
49+
50+
**Cause:** Same root cause as the demos above — these packages reference removed/renamed APIs (`Graph.explain`, `memoryStorage`, `StorageTier`, `fileStorage`) that drifted during Phase 4+ refactors. Additionally `cli` imports from `@graphrefly/mcp-server`, which isn't on npm yet — chicken/egg.
51+
52+
**Workaround in place:** Both packages marked `"private": true` in their `package.json`. Changesets respects the flag and skips publishing them. They still build via `pnpm build` for local dev (no publish guard there), but `prepublishOnly` is gated behind the privacy flag.
53+
54+
**Decision deferred to:** same window as the demo migration (post-Rust-port). Restoration:
55+
1. Migrate `cli/src/dispatch.ts` and `mcp-server/src/{session,tools}.ts` to current API (Graph methods, current storage symbols).
56+
2. Decide cli↔mcp-server dependency model:
57+
- Option A: cli imports mcp-server via `workspace:*` (already does), publish cli AFTER mcp-server lands on npm.
58+
- Option B: cli inlines the mcp-server entry it needs, drops the dependency.
59+
3. Remove `"private": true` from both `package.json` files.
60+
4. Restore the `NOT YET PUBLISHED — see docs/known-issues.md` text in `description` to the original.
61+
5. Set up npm trusted-publisher config for both at `https://www.npmjs.com/package/<name>/access`.
62+
6. Push a changeset that bumps both → next release publishes them.
63+
7. Delete this section from `known-issues.md`.

packages/cli/package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
{
22
"name": "@graphrefly/cli",
33
"version": "0.0.2",
4-
"description": "Command-line interface for GraphReFly — stateless shell over the surface layer (describe, explain, reduce, snapshot, run an MCP server).",
4+
"private": true,
5+
"description": "Command-line interface for GraphReFly — stateless shell over the surface layer (describe, explain, reduce, snapshot, run an MCP server). NOT YET PUBLISHED — see docs/known-issues.md.",
56
"license": "MIT",
67
"repository": {
78
"type": "git",

packages/mcp-server/package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
{
22
"name": "@graphrefly/mcp-server",
33
"version": "0.0.2",
4-
"description": "Model Context Protocol server for GraphReFly — expose graph operations (create, describe, observe, explain, reduce, snapshot) as MCP tools.",
4+
"private": true,
5+
"description": "Model Context Protocol server for GraphReFly — expose graph operations (create, describe, observe, explain, reduce, snapshot) as MCP tools. NOT YET PUBLISHED — see docs/known-issues.md.",
56
"license": "MIT",
67
"mcpName": "io.github.graphrefly/mcp-server",
78
"repository": {

0 commit comments

Comments
 (0)