Contributor Guide
- Prerequisites
- Setup
- Coding agents
- Project Structure
- Development Workflow
- Testing & Linting
- Code Style
- Branching & Commits
- CI/CD
- Architecture Overview
Prerequisites
- Node.js v20 or later
- corepack on
PATHand enabled.corepack --versionmust resolve in the shell you runyarn installfrom — the build scripts invokecorepack yarn …directly. If it printscommand not found(Volta and a few other version managers don't ship acorepackshim), install it first:bashStandard Node distributions already includenpm install -g corepack@latest corepack enablecorepack;corepack enableis enough. - VS Code
The webview and docs dev scripts (standalone browser preview) use portless to serve each app under a stable, worktree-aware https://<worktree>.<app>.localhost URL — no port numbers to remember or collide. portless is a pinned dev dependency installed by yarn install, so no global install is required. The slug is derived by portless from the git worktree; never hand-build it.
Its HTTPS proxy daemon is a one-time per-machine step (needs sudo once):
npx portless service installEach app keeps the split dev / dev:app scripts: dev is just the portless entrypoint, and dev:app is the real Vite/VitePress command (which non-portless users can run directly). portless.json names the app and points dev at dev:app so it doesn't recurse.
Setup
yarn installCoding agents
Claude Code and Codex share the same project instructions and skills:
| Entry point | Shared source |
|---|---|
AGENTS.md (Codex), CLAUDE.md (Claude Code) | AGENTS.md |
.agents/skills/ (Codex), .claude/skills/ (Claude Code) | .agents/skills/ |
CLAUDE.md and .claude/skills/ are Git symlinks to the shared sources. Edit AGENTS.md and .agents/skills/ to keep both clients in sync. Invoke a skill with /architecture in Claude Code or $architecture in Codex. The same convention applies to adr, commit, and the other repository skills. Claude's legacy /commit command wrapper also points to the shared commit skill.
.claude/settings.json contains only Claude settings: automatic attribution is disabled and the existing Playwright plugin tools are pre-approved. Codex does not read this file. The rule against agent attribution in commits and PRs lives in the shared instructions and applies to both clients. Keep personal Claude overrides in .claude/settings.local.json (gitignored). When updating an older checkout, move any local Claude overrides from the former shared agent directory to .claude/settings.local.json; .claude/ is now a real directory with only its skills symlinked to the shared source.
Browser tools
Browser testing uses Microsoft's Playwright MCP server. Configure it once for each client you use. Reuse an existing Playwright installation if it already exposes the browser tools; no duplicate server is needed. The following user-scoped setup pins the server version and gives each session an isolated browser profile for worktree use:
# Claude Code: stored in ~/.claude.json
claude mcp add --scope user playwright -- npx -y @playwright/mcp@0.0.80 --isolated
# Codex: stored in ~/.codex/config.toml
codex mcp add playwright -- npx -y @playwright/mcp@0.0.80 --isolatedThe repo's Claude permission entries target the existing plugin's tool names; they do not install a server or pre-approve a separately named playwright server. Use each client's normal tool approval controls for that server.
For team-managed project configuration, Claude uses root .mcp.json, while Codex uses .codex/config.toml in a trusted project. This repo documents the user-scoped setup above and does not install MCP servers during yarn install. Conductor loads each agent's own MCP configuration; no additional Conductor MCP format is needed. See the Conductor MCP reference, Claude MCP reference, and Codex MCP reference.
Verify a new session
- Restart the client after changing instructions or tool configuration. Ask it to summarize the repository instructions: it should identify the shared source,
corepack yarn, the ADR rule, and the commit conventions. - Check that all ten repository skills are available, including
architecture,commit, andbpmn-browser-testing. Use/skillsor$in Codex and the/menu in Claude Code. Merely checking the commit skill should not commit anything. - Run
claude mcp listorcodex mcp listand check Playwright's status in the session. In Conductor, refresh MCP status and start a new session if needed. - Start
corepack yarn workspace @miragon/bpmn-modeler-webview serve, open the URL printed by Vite, and ask the agent to take a screenshot and inspect the palette with a browser snapshot. Follow any missing-browser installation guidance returned by the server.
Browser tool names and prefixes vary by client/plugin. The browser-testing skill uses the tool schemas available in the session, including the exposed Playwright-code tool for page.mouse interactions.
Project Structure
This is a Yarn 4 workspace monorepo containing a single VS Code extension (Camunda Modeler) and its supporting packages:
| Workspace | Path | Description |
|---|---|---|
vs-code-bpmn-modeler | apps/vscode-plugin | VS Code extension host (Node/Webpack) |
@miragon/bpmn-modeler-webview | apps/bpmn-webview | BPMN editor UI (Vite/browser) |
@miragon/dmn-modeler-webview | apps/dmn-webview | DMN editor UI (Vite/browser) |
@miragon/bpmn-modeler-shared | libs/shared | Shared message types and utilities |
Workspace dependencies
Every workspace declares its own runtime and build dependencies in its own package.json. The root package.json only carries cross-cutting tooling (eslint, prettier, npm-run-all, typescript). This lets CI install just the tree it needs via yarn workspaces focus:
# Modeler-only tree (no Theia, no native-keymap, no apt-step required)
yarn workspaces focus bpmn-modeler vs-code-bpmn-modeler @miragon/bpmn-modeler-webview \
@miragon/dmn-modeler-webview @miragon/bpmn-modeler-deployment-webview @miragon/bpmn-modeler-shared @miragon/bpmn-modeler-append-menu \
@miragon/bpmn-modeler-clipboard @miragon/bpmn-modeler-i18n \
@miragon/bpmn-modeler-element-template-chooser
# Just the docs site
yarn workspaces focus bpmn-modeler docs
# Full repo (needed for the standalone Theia app)
yarn installThe standalone app (apps/standalone) pulls Theia + native-keymap, whose node-gyp postinstall needs libx11-dev libxkbfile-dev libsecret-1-dev on Linux — which is why only the full-install workflow (build.yml) runs the apt-step.
Development Workflow
Build
# Build everything (libs → webviews + plugin in parallel)
yarn build
# Build only the shared libraries
yarn build:libsWatch mode
# Rebuild all workspaces on change (feeds the F5 Extension Host)
yarn watchDocs site
yarn docs:devOpens the VitePress docs site in your browser.
Run the extension in VS Code
- Open the repository root in VS Code.
- Run
yarn watchto start watch mode. - Open the Run and Debug panel and select "Run vscode-plugin".
- Press F5 to launch the Extension Development Host.
To reload the extension host after a change, press Cmd+R (macOS) or Ctrl+R ( Windows/Linux).
Target a single workspace
yarn workspace vs-code-bpmn-modeler build
yarn workspace @miragon/bpmn-modeler-webview buildPreview the BPMN webview in a plain browser
The BPMN webview can run standalone against a mocked VS Code host. This avoids reloading the Extension Development Host while iterating on webview UI.
yarn dev:bpmn-webviewThis launches a Vite dev server via portless; the URL is printed to stdout when the server starts.
A URL query parameter selects what the mock serves:
| URL | What renders |
|---|---|
/ (or ?mode=modeler) | Full editable Camunda modeler with a hardcoded sample diagram — matches the production modeler experience. |
/?mode=diff-before | Readonly before (left) pane of a diff view, with highlights for removed / changed / moved elements. |
/?mode=diff-after | Readonly after (right) pane, with highlights for added / changed / moved elements. |
The diff modes run bpmn-js-differ against two fixture XMLs (apps/bpmn-webview/src/app/__fixtures__/mock-diff.ts) so highlights reflect the real differ's output. All mock code and its dependencies are gated on NODE_ENV === "development" and tree-shaken out of the production webview bundle.
Testing & Linting
# Run all tests (includes coverage by default)
yarn test
# Run a single test file
yarn test apps/vscode-plugin/src/shared/domain/BpmnDocument.spec.ts
# Lint
yarn lintCoverage reports are uploaded to Codecov on CI.
Performance harness
Large models are generated on demand rather than committed. Everything lives in scripts/perf/.
Generate a large model
yarn perf:gen-model 100 50 > large-5000.bpmn # rows × perRow flow nodesThis produces a flat Camunda 7 process of laid-out chains that mix user tasks, asyncAfter service tasks and gateways. The presets are 500 (10 50), 2,000 (40 50) and 5,000 (100 50) nodes. largeBpmnModel.mjs exports the same generator for scripts and specs, together with counts such as the expected number of transaction boundaries.
Benchmark the production webview
yarn build:libs && yarn build:bpmn-webview
yarn perf:webview # full matrix, 5 runs per cell
yarn perf:webview --sizes 5000 --lint in-page --locales de --runs 3
yarn perf:webview --json results.json # also write raw samplesThe bench serves the production bundle from dist/webview-staging/bpmn-webview/ and opens it in headless Chromium. It uses its own host shim, because the dev MockHost is compiled out of production builds. The shim answers the webview's requests asynchronously, in the same order as the real host: file (carrying the initial locale), lint config, settings, then LanguageQuery.
- Lint
off: the host replies withBpmnLintDisabledQuery. - Lint
in-page: the host replies with a payload-freeBpmnlintInPageQuery, the zero-config VS Code default.
For every cell it prints a Markdown table (median and min–max) with these columns:
- Open: busy until: the time from navigation start to the end of the last long task, once the main thread has been quiet for 7 s. The window is longer than the longest lint deferral, so a debounced lint pass after the last edit is still measured. Deferral time counts towards the metric.
- Open: longest task: the longest single main-thread block while opening.
- Edit burst: the same two metrics after five real mouse drags on the canvas. The canvas is first zoomed in so that the drags hit their elements.
The numbers are a lower bound. They include no host IPC, no JCEF and no extension-host work, so compare runs only on the same machine. If the webview never clears its busy state, the run fails and lists the host messages the shim left unanswered. This usually means the protocol changed and the shim in bench-webview.mjs needs updating.
Benchmark individual lint rules
yarn perf:lint # no-overlapping-elements, no-bpmndi
yarn perf:lint --rules label-required --sizes 5000 --runs 10
yarn perf:lint --json results.json # also write raw samplesThe bench runs in Node, without a browser or a build. It parses each preset model once, then times a single-rule Linter pass per run. It prints one Markdown row per rule, with the median and min–max per model size, plus the CPU, the core count and the Node version.
Rules resolve through bpmnlint's NodeResolver, so --rules accepts bpmnlint's built-in rule names only. Plugin rules such as those in @miragon/bpmnlint-plugin-rules need a different resolver: raw Node cannot import that ESM bundle because of its extensionless imports.
Counter spec
packages/bpmn-modeler/src/largeModelCounters.browser.spec.ts runs in CI as part of the Chromium browser project. It opens a generated 2k-node model through createModeler, following the package's in-page lint path, the webview's external → startInPageLinting handback path, and an external path with host-pushed results.
For open, re-import, an unchanged relint and a five-edit burst, it pins these counters:
- imports
- full lint rule runs
- transaction-boundary overlays that were added and that are present
- lint overlays that were added and removed
The pinned values record the current behaviour, not targets. A change that removes redundant work updates the matching value in the same PR, so the improvement shows up in the diff and a regression fails CI.
Code Style
| Tool | Configuration | Key rules |
|---|---|---|
| EditorConfig | .editorconfig | 4-space indent, LF line endings, max 89 chars |
| Prettier | .prettierrc | Double quotes, trailing commas, arrow parens always |
| ESLint | eslint.config.mjs | TypeScript strict |
Prettier and ESLint are enforced by the lint step in CI.
Branching & Commits
Branching model
Commit messages
Use semantic commit messages scoped to the affected workspace:
feat(bpmn): add token simulation toolbar
fix(dmn): correct decision table rendering
chore(shared): update message type definitionsCommon types: feat, fix, refactor, chore, docs, test.
CI/CD
| Workflow | Trigger | Purpose |
|---|---|---|
| Build | every push / PR | lint → test → build, full install (apt-step for Theia native modules) |
| PR Labeler | PR opened / updated | auto-labels PRs by changed workspace |
| **Prepare Release *** | manual (workflow_dispatch) | bump version, sanity build, commit, tag, create GitHub Release |
| **Publish *** | release: published (or workflow_dispatch + dry-run) | build artefact, attach to release, push to Marketplace / GitHub Release |
| Deploy Docs | release: published / manual | VitePress build + GitHub Pages deploy |
There are two prepare-* and two publish-* workflows — one pair per artefact (VS Code extension, standalone macOS app). See Release process for the operational guide and the pipeline flow diagram.
Architecture Overview
The extension is organised by feature with plain constructor wiring — no DI framework. Each feature folder owns the four classic layers as subfolders, and cross-feature use goes through the feature's index.ts barrel.
apps/vscode-plugin/src/
main.ts # Activation: build shared deps, then call each feature's register()
composition/ # One register(context, deps) per feature — the wiring root
shared/ # Cross-feature substrate: domain/ service/ infrastructure/
# (EditorSessionStore, VsCode* adapters, WebviewMessageRouter, …)
modeler/
editor-session/ # Generic ModelerEditorController + EditorSessionParticipant
bpmn/ dmn/ # domain/ service/ controller/ infrastructure/ index.ts
diff/ deployment/ scriptTask/ navigation/ migration/ # same per-feature layoutThe layer + feature-isolation boundaries are enforced in CI by apps/vscode-plugin/src/architecture.spec.ts (ArchUnitTS). See the Architecture overview for the full model.
Key design decisions:
- Echo prevention: each open editor gets content-aware
ModelerSessionguards that block only the matchingonDidChangeTextDocumentecho from an extension write. - Element template discovery: convention-based — no project config file needed. Templates are resolved under
<configFolder>/element-templates/walking up from the BPMN file to the workspace root. - Webview communication:
postMessagewith typed message contracts defined inlibs/shared.
See AGENTS.md in the repository root for the full architectural reference.