Skip to content

Commit fa9af94

Browse files
anerlianerli
committed
rename to pi pkg to pi-extension
1 parent 3fb35af commit fa9af94

11 files changed

Lines changed: 31 additions & 31 deletions

File tree

‎bun.lock‎

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎cli/src/commands/connections.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ describe("connections command contract", () => {
5656
connection: {
5757
companion: Option.some({
5858
name: "Magnitude for Pi",
59-
source: "npm:@magnitudedev/[email protected]",
59+
source: "npm:@magnitudedev/pi-extension@0.0.1",
6060
securityNotice: "Pi extensions execute with your user permissions.",
6161
status: "installed",
6262
activationInstructions: Option.some("Restart existing Pi sessions or run /reload to activate the extension."),

‎cli/src/features/model-setup/harness.test.tsx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ const destinations = [
3939
skillRequired: true,
4040
companion: {
4141
name: "Magnitude for Pi",
42-
source: "npm:@magnitudedev/[email protected]",
42+
source: "npm:@magnitudedev/pi-extension@0.0.1",
4343
securityNotice: "Pi extensions execute with your user permissions.",
4444
},
4545
},
@@ -138,7 +138,7 @@ describe("harness chooser layout", () => {
138138
act(() => keyboard.handler?.(keyEvent("down")))
139139
await act(view.renderOnce)
140140
const frame = view.captureCharFrame()
141-
expect(frame).toContain("Magnitude will install and enable Magnitude for Pi (npm:@magnitudedev/[email protected]).")
141+
expect(frame).toContain("Magnitude will install and enable Magnitude for Pi (npm:@magnitudedev/pi-extension@0.0.1).")
142142
expect(frame).toContain("Pi extensions execute with your user permissions.")
143143
expect(frame).toContain("[x] Magnitude skill required for agent model management")
144144
} finally {

‎cli/src/harness-connections/connectors/pi-package-state.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { Schema } from "effect"
33
export const PiPackageSourceSchema = Schema.NonEmptyString.pipe(Schema.brand("PiPackageSource"))
44
export type PiPackageSource = typeof PiPackageSourceSchema.Type
55

6-
const fields = { identity: Schema.Literal("@magnitudedev/pi"), source: PiPackageSourceSchema }
6+
const fields = { identity: Schema.Literal("@magnitudedev/pi-extension"), source: PiPackageSourceSchema }
77
/** A receipt records only the package and fields Magnitude is entitled to undo. */
88
export const PiCompanionStateSchema = Schema.Union(
99
Schema.Struct({ ...fields, ownership: Schema.Literal("magnitude") }),

‎cli/src/harness-connections/connectors/pi-package.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ import {
1818
} from "./pi-settings"
1919

2020
export const PI_COMPANION_PACKAGE_NAME = "Magnitude for Pi"
21-
export const PI_COMPANION_PACKAGE_IDENTITY = "@magnitudedev/pi"
22-
export const PI_COMPANION_PACKAGE_SOURCE = PiPackageSourceSchema.make("npm:@magnitudedev/[email protected]")
21+
export const PI_COMPANION_PACKAGE_IDENTITY = "@magnitudedev/pi-extension"
22+
export const PI_COMPANION_PACKAGE_SOURCE = PiPackageSourceSchema.make("npm:@magnitudedev/pi-extension@0.0.1")
2323
export const PI_COMPANION_EXTENSION_PATH = "dist/magnitude.js"
2424
const SUPPORTED_PACKAGE_VERSION = "0.0.1"
2525

‎cli/src/harness-connections/service.test.ts‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -125,13 +125,13 @@ const readYaml = (source: string): unknown => parseDocument(source).toJS()
125125
const testPiCompanion: HarnessCompanionPackage = {
126126
description: {
127127
name: "Magnitude for Pi",
128-
source: "npm:@magnitudedev/[email protected]",
128+
source: "npm:@magnitudedev/pi-extension@0.0.1",
129129
securityNotice: "Pi extensions execute with your user permissions.",
130130
},
131131
activationInstructions: Option.some("Restart existing Pi sessions or run /reload to activate the extension."),
132132
reconcile: ({ previous }) => Effect.succeed({
133133
state: Option.getOrElse(previous, () => ({
134-
identity: "@magnitudedev/pi",
134+
identity: "@magnitudedev/pi-extension",
135135
source: PI_COMPANION_PACKAGE_SOURCE,
136136
ownership: "magnitude" as const,
137137
})),
@@ -208,8 +208,8 @@ const storedSource = isAbsolute(source) ? relative(dirname(settingsPath), source
208208
const equalSource = (value) => value === source || (!value.startsWith("npm:") && resolve(dirname(settingsPath), value) === resolve(dirname(settingsPath), source))
209209
if (command === "install") {
210210
settings.packages = packages.some((entry) => equalSource(typeof entry === "string" ? entry : entry.source)) ? packages : [...packages, storedSource]
211-
const root = source.startsWith("npm:") ? resolve(dirname(settingsPath), "npm/node_modules/@magnitudedev/pi") : source
212-
await Bun.write(resolve(root, "package.json"), JSON.stringify({ name: "@magnitudedev/pi", version: "0.0.1", pi: { extensions: ["./dist/magnitude.js"] } }))
211+
const root = source.startsWith("npm:") ? resolve(dirname(settingsPath), "npm/node_modules/@magnitudedev/pi-extension") : source
212+
await Bun.write(resolve(root, "package.json"), JSON.stringify({ name: "@magnitudedev/pi-extension", version: "0.0.1", pi: { extensions: ["./dist/magnitude.js"] } }))
213213
await Bun.write(resolve(root, "dist/magnitude.js"), "export default () => {}")
214214
}
215215
else if (command === "remove") settings.packages = packages.filter((entry) => {
@@ -432,8 +432,8 @@ describe("HarnessConnector contract and registry", () => {
432432
describe("Pi companion package lifecycle", () => {
433433
it.each([
434434
'{"packages":{}}',
435-
'{"packages":[{"source":"npm:@magnitudedev/[email protected]","extensions":[42]}]}',
436-
'{"packages":[{"source":"npm:@magnitudedev/[email protected]","autoload":"false"}]}',
435+
'{"packages":[{"source":"npm:@magnitudedev/pi-extension@0.0.1","extensions":[42]}]}',
436+
'{"packages":[{"source":"npm:@magnitudedev/pi-extension@0.0.1","autoload":"false"}]}',
437437
'{"packages":[{"extensions":[]}]}',
438438
])("rejects invalid settings before native package operations: %s", async (settings) => {
439439
await Effect.runPromise(Effect.scoped(Effect.gen(function* () {
@@ -473,8 +473,8 @@ describe("Pi companion package lifecycle", () => {
473473
yield* writeFixtures({
474474
...initialFiles(paths),
475475
[paths.piSettings]: settings,
476-
[`${root}/pi/npm/node_modules/@magnitudedev/pi/package.json`]: stringifyJson({ name: "@magnitudedev/pi", version: "0.0.1", pi: { extensions: ["./dist/magnitude.js"] } }),
477-
[`${root}/pi/npm/node_modules/@magnitudedev/pi/dist/magnitude.js`]: "export default () => {}",
476+
[`${root}/pi/npm/node_modules/@magnitudedev/pi-extension/package.json`]: stringifyJson({ name: "@magnitudedev/pi-extension", version: "0.0.1", pi: { extensions: ["./dist/magnitude.js"] } }),
477+
[`${root}/pi/npm/node_modules/@magnitudedev/pi-extension/dist/magnitude.js`]: "export default () => {}",
478478
})
479479
const executable = yield* writeFakePiExecutable(paths)
480480
const service = yield* makeHarnessConnectionService({ paths, detect: () => Effect.succeed(Option.some({ executable })), resolveModels: Effect.succeed(models), installStartup: Effect.void })
@@ -544,8 +544,8 @@ describe("Pi companion package lifecycle", () => {
544544
yield* writeFixtures({
545545
...initialFiles(paths),
546546
[paths.piSettings]: stringifyJson({ theme: "original", packages: [existing] }),
547-
[`${root}/pi/npm/node_modules/@magnitudedev/pi/package.json`]: stringifyJson({ name: "@magnitudedev/pi", version: "0.0.1", pi: { extensions: ["./dist/magnitude.js"] } }),
548-
[`${root}/pi/npm/node_modules/@magnitudedev/pi/dist/magnitude.js`]: "export default () => {}",
547+
[`${root}/pi/npm/node_modules/@magnitudedev/pi-extension/package.json`]: stringifyJson({ name: "@magnitudedev/pi-extension", version: "0.0.1", pi: { extensions: ["./dist/magnitude.js"] } }),
548+
[`${root}/pi/npm/node_modules/@magnitudedev/pi-extension/dist/magnitude.js`]: "export default () => {}",
549549
})
550550
const executable = yield* writeFakePiExecutable(paths)
551551
const service = yield* makeHarnessConnectionService({ paths, detect: () => Effect.succeed(Option.some({ executable })), resolveModels: Effect.succeed(models), installStartup: Effect.void })
@@ -569,7 +569,7 @@ describe("Pi companion package lifecycle", () => {
569569
const executable = yield* writeFakePiExecutable(paths)
570570
const service = yield* makeHarnessConnectionService({ paths, detect: () => Effect.succeed(Option.some({ executable })), resolveModels: Effect.succeed(models), installStartup: Effect.void })
571571
yield* service.connect(HarnessIdSchema.make("pi"), { model: Option.none() })
572-
const entrypoint = `${root}/pi/npm/node_modules/@magnitudedev/pi/dist/magnitude.js`
572+
const entrypoint = `${root}/pi/npm/node_modules/@magnitudedev/pi-extension/dist/magnitude.js`
573573
yield* fs.remove(entrypoint)
574574
yield* service.sync(HarnessIdSchema.make("pi"))
575575
expect(yield* fs.exists(entrypoint)).toBe(true)
@@ -582,9 +582,9 @@ describe("Pi companion package lifecycle", () => {
582582
const fs = yield* FileSystem.FileSystem
583583
const root = yield* fs.makeTempDirectoryScoped({ prefix: "magnitude-pi-incompatible-" })
584584
const paths = fixturePaths(root)
585-
const settings = stringifyJson({ packages: ["npm:@magnitudedev/[email protected]"] })
585+
const settings = stringifyJson({ packages: ["npm:@magnitudedev/pi-extension@9.0.0"] })
586586
yield* writeFixtures({ ...initialFiles(paths), [paths.piSettings]: settings,
587-
[`${root}/pi/npm/node_modules/@magnitudedev/pi/package.json`]: stringifyJson({ name: "@magnitudedev/pi", version: "9.0.0", pi: { extensions: ["./dist/magnitude.js"] } }),
587+
[`${root}/pi/npm/node_modules/@magnitudedev/pi-extension/package.json`]: stringifyJson({ name: "@magnitudedev/pi-extension", version: "9.0.0", pi: { extensions: ["./dist/magnitude.js"] } }),
588588
})
589589
const executable = yield* writeFakePiExecutable(paths)
590590
const service = yield* makeHarnessConnectionService({ paths, detect: () => Effect.succeed(Option.some({ executable })), resolveModels: Effect.succeed(models), installStartup: Effect.void })
@@ -771,8 +771,8 @@ describe("Pi companion package lifecycle", () => {
771771
const root = yield* fs.makeTempDirectoryScoped({ prefix: "magnitude-pi-package-existing-" })
772772
const paths = fixturePaths(root)
773773
yield* writeFixtures({
774-
[`${root}/pi/npm/node_modules/@magnitudedev/pi/package.json`]: stringifyJson({ name: "@magnitudedev/pi", version: "0.0.1", pi: { extensions: ["./dist/magnitude.js"] } }),
775-
[`${root}/pi/npm/node_modules/@magnitudedev/pi/dist/magnitude.js`]: "export default () => {}",
774+
[`${root}/pi/npm/node_modules/@magnitudedev/pi-extension/package.json`]: stringifyJson({ name: "@magnitudedev/pi-extension", version: "0.0.1", pi: { extensions: ["./dist/magnitude.js"] } }),
775+
[`${root}/pi/npm/node_modules/@magnitudedev/pi-extension/dist/magnitude.js`]: "export default () => {}",
776776
})
777777
const existing = {
778778
source: PI_COMPANION_PACKAGE_SOURCE,

‎info/harness-integration-feasibility.md‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -92,10 +92,10 @@ This preserves Pi's message, tool-call, usage, and error semantics while giving
9292

9393
### Packaging and lifecycle
9494

95-
Publish an npm package such as `@magnitudedev/pi` with the `pi-package` keyword and `pi.extensions` entry. Pi's documented lifecycle is:
95+
Publish an npm package such as `@magnitudedev/pi-extension` with the `pi-package` keyword and `pi.extensions` entry. Pi's documented lifecycle is:
9696

97-
- Install: `pi install npm:@magnitudedev/pi`
98-
- Remove: `pi remove npm:@magnitudedev/pi`
97+
- Install: `pi install npm:@magnitudedev/pi-extension`
98+
- Remove: `pi remove npm:@magnitudedev/pi-extension`
9999
- Reload the running process with `/reload`, or restart it.
100100

101101
Pi extensions execute with the user's permissions, so CLI failures should be surfaced explicitly and never treated as successful state changes.

‎integrations/pi/README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ magnitude connections add pi
1111
```
1212

1313
Restart Pi or run `/reload` after connecting. `PI_CODING_AGENT_DIR`, if set, is honored.
14-
Installing just `pi install npm:@magnitudedev/[email protected]` does not configure provider models,
14+
Installing just `pi install npm:@magnitudedev/pi-extension@0.0.1` does not configure provider models,
1515
install the Magnitude CLI, start its service, or install its agent skill; the connection command
1616
does all connection configuration and can be run safely after a standalone package install.
1717

‎integrations/pi/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"name": "@magnitudedev/pi",
2+
"name": "@magnitudedev/pi-extension",
33
"version": "0.0.1",
44
"description": "Magnitude model controls and inference progress for Pi",
55
"type": "module",

‎packages/client-common/src/local-models/setup.test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -817,7 +817,7 @@ describe("OnboardingModelSetup", () => {
817817
const connect = vi.fn(() => Effect.succeed({
818818
companion: Option.some({
819819
name: "Magnitude for Pi",
820-
source: "npm:@magnitudedev/[email protected]",
820+
source: "npm:@magnitudedev/pi-extension@0.0.1",
821821
securityNotice: "Pi extensions execute with your user permissions.",
822822
status: "installed" as const,
823823
activationInstructions: Option.some("Restart existing Pi sessions or run /reload to activate the extension."),
@@ -834,7 +834,7 @@ describe("OnboardingModelSetup", () => {
834834
connected: false,
835835
companion: {
836836
name: "Magnitude for Pi",
837-
source: "npm:@magnitudedev/[email protected]",
837+
source: "npm:@magnitudedev/pi-extension@0.0.1",
838838
securityNotice: "Pi extensions execute with your user permissions.",
839839
},
840840
}]),

0 commit comments

Comments
 (0)