Skip to content

ci: dogfood interactive AppMap before/after PR comparisons - #1

Draft
tryingET wants to merge 9 commits into
mainfrom
agent/dogfood-appmap-comparison
Draft

ci: dogfood interactive AppMap before/after PR comparisons#1
tryingET wants to merge 9 commits into
mainfrom
agent/dogfood-appmap-comparison

Conversation

@tryingET

@tryingET tryingET commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Purpose

Use appmap-node as the end-to-end dogfood project for the frozen AppMap before/after comparison contract.

The workflow builds exact base/head revisions, records deterministic AppMaps, generates portable appmap.comparison schema-v1 bundles, opens the real generated artifact in the pinned VS Code extension host, and verifies the review publisher and skill contract.

Final acceptance run

Dogfood AppMap PR comparison — run 32226024667

All three PR checks passed:

  • ✅ Record and compare base/head behavior
  • ✅ Build and exercise comparison viewer
  • ✅ Verify review artifact publisher

Verified outcomes:

  • comparison model contract: 12 tests passed
  • CLI producer: 2 tests passed
  • exact base/head builds completed
  • four source AppMaps sanitized
  • clean control: 0 behavioral changes
  • visual scenario: exactly 1 call-added authorize change
  • deterministic, non-positional chg_<hash> IDs
  • structural changes without event IDs retained through elementIds
  • real generated bundle opened in VS Code Electron
  • dogfood VSIX packaged
  • pinned skill and publisher harness passed

Exact dependency lock

  • appmap-node base: e52b6ce778ffecba9a0c713655dcad112bb828a8
  • appmap-node head: 166ce2797f17346e4a978748a34639b8abfe023c
  • appmap-js: aab690a5c4c01802551993718b508718c1139c11
  • vscode-appland: cfd3a0fef35b016cb0af20ee02ae46fd35414069
  • review-action: 67060904f03c0c8a1492e610f845eefd29e53a78
  • skills: 167db8d4b7eb56086c74a9ef9ffe7b2d3c8ab627

Artifacts

  • appmap-pr-comparison-schema-v1 — artifact 9355642181, SHA-256 9e7e2d34d75dcae27a6a60f7b9d22d90daa04233af4a2358ddda2ae38b6ffac0
  • appmap-comparison-schema-v1-vsix — artifact 9355691022, SHA-256 c845728666068d97d58864bb091e14bf0cc33ed314097a2d776cc00745429379

The data artifact contains generated bundles, four sanitized AppMaps, JSON Schema, compatibility documentation, schema changelog, all five conformance fixtures, and the exact revision lock.

Where to see the run

The workflow is attached natively to this PR and is visible under its Checks tab. GitHub job summaries and artifact downloads live on the workflow-run page; they are not automatically embedded into the PR Conversation view.

Related work

  • Producer/contract: tryingET/appmap-js#1 and getappmap/appmap-js#2388
  • Viewer: tryingET/vscode-appland#1
  • Publisher: tryingET/review-action#1
  • Skill: tryingET/skills#1

Kept draft as a migration and integration record.

tryingET commented Aug 18, 2026

Copy link
Copy Markdown
Owner Author

Final pinned schema-v1 dogfood complete

Dogfood AppMap PR comparison run 32226024667 is fully green across all three jobs and is the acceptance run for the frozen comparison contract.

Exact dependency lock

  • appmap-node base: e52b6ce778ffecba9a0c713655dcad112bb828a8
  • appmap-node head: 166ce2797f17346e4a978748a34639b8abfe023c
  • appmap-js: aab690a5c4c01802551993718b508718c1139c11
  • vscode-appland: cfd3a0fef35b016cb0af20ee02ae46fd35414069
  • review-action: 67060904f03c0c8a1492e610f845eefd29e53a78
  • skills: 167db8d4b7eb56086c74a9ef9ffe7b2d3c8ab627

The workflow verifies every checked-out commit before use.

Verified outcomes

  • comparison contract model tests: 12 passed
  • CLI producer tests: 2 passed
  • exact base/head builds completed
  • all four source AppMaps sanitized
  • control comparison: 0 behavioral changes
  • visual comparison: exactly 1 call-added authorize change
  • deterministic, non-positional chg_<hash> IDs
  • structural actions without event IDs remain represented through elementIds
  • frozen JSON Schema, docs, changelog, fixtures, and contract lock embedded in the artifact
  • the real generated bundle opened in the pinned VS Code Electron extension host
  • expected authorize change resolved through the schema-v1 change model
  • dogfood VSIX packaged successfully
  • pinned skill and review-action publisher harness passed

Final artifacts

  • appmap-pr-comparison-schema-v1 — artifact 9355642181, SHA-256 9e7e2d34d75dcae27a6a60f7b9d22d90daa04233af4a2358ddda2ae38b6ffac0
  • appmap-comparison-schema-v1-vsix — artifact 9355691022, SHA-256 c845728666068d97d58864bb091e14bf0cc33ed314097a2d776cc00745429379

The data artifact is now a reproducible migration input: it contains generated bundles, sanitized recordings, the contract itself, all five conformance fixtures, and the exact revision lock.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant