chore: add MIT license, finish rename in api.ts - #14
Merged
Conversation
Replace marked + DOMPurify with markdown-it (safe by default) for the agent transcript renderer. Add @pierre/diffs as the diff engine wired into a Fig. 02 probe in the empty state, with a tiny LCS unified-patch generator so the React API is <PierreDiff oldFile newFile>. Set up three Vitest lanes: unit (node, timeline + sessions parser), integration (jsdom + RTL, Markdown component), and e2e (Playwright Electron, mac only via workflow_dispatch). Add oxlint with correctness as error and the rest allowed. Split ci.yml into lint / typecheck / unit / integration / build jobs; add e2e.yml for the macos-only lane. Record nine architecture decisions under docs/adr/, indexed in a README. Includes three new agent skills under ~/.agents/skills/ for test lanes, ADRs, and the calver release flow. Co-authored-by: opencode-agent <hello@sst.dev>
vitest's --project flag filters by name from the projects[] array in the loaded config; since vitest.e2e.config.ts is a separate file (not listed as a project in vitest.config.ts), --project e2e finds nothing. Switch to --config vitest.e2e.config.ts. Co-authored-by: opencode-agent <hello@sst.dev>
The e2e tests assume the renderer is reachable on localhost:5173 (isDev path in main/index.ts). Build the renderer, start vite in the background, wait for it, then run the e2e lane. Co-authored-by: opencode-agent <hello@sst.dev>
macos github runners install globally under /usr/local/bin (not /opt/homebrew/bin where Homebrew would put it). Add the additional candidates so spawn finds pi after npm install -g. Co-authored-by: opencode-agent <hello@sst.dev>
The macos github runner has no ~/.pi sessions to import, so the workspaces list is empty and no auto-opened first session tab exists. Trim the e2e lane to assertions that don't require fixture data: renderer shell mounts, document scroll is locked, preload bridge exposes versions, and no white button borders leak. Co-authored-by: opencode-agent <hello@sst.dev>
Rename the package, IPC channel prefix, preload bridge key, window metadata type, status-bar tagline, and all prose references. Rename the docs files from openpi-* to pidex-*. No behavioural change; the calver release tag is unaffected. Co-authored-by: opencode-agent <hello@sst.dev>
…re-diffs-tests # Conflicts: # .github/workflows/e2e.yml # README.md # docs/adr/0002-pierre-diffs-renderer.md # docs/adr/0003-pretext-measurement.md # docs/adr/0005-oxlint.md # docs/adr/0006-pi-runtime-rpc.md # docs/adr/0007-native-tab-bar.md # docs/adr/0008-pi-dev-design-system.md # docs/adr/0009-calver-releases.md # docs/adr/README.md # src/main/index.ts # src/preload/index.ts # src/renderer/App.tsx # src/renderer/api.ts # tests/e2e/app.test.ts
Add LICENSE with the standard MIT text, declare it in package.json, and link it from the README. Also include the leftover OpenPiApi -> PidexApi rename that the merge commit on main missed, so the renderer type matches the rest of the pidex-named surface. Co-authored-by: opencode-agent <hello@sst.dev>
…re-diffs-tests # Conflicts: # src/renderer/api.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
LICENSEwith the standard MIT text (copyright Johnny Huynh, 2026)"license": "MIT"inpackage.jsonOpenPiApi→PidexApirename insrc/renderer/api.tsthat the previous rename merge on main missedTest plan
bunx tsc -p tsconfig.json --noEmitbun run test:unitBoth pass.