Skip to content

chore: support opening .trace files via .link indirection#39975

Merged
pavelfeldman merged 1 commit intomicrosoft:mainfrom
pavelfeldman:trace-link-file
Mar 31, 2026
Merged

chore: support opening .trace files via .link indirection#39975
pavelfeldman merged 1 commit intomicrosoft:mainfrom
pavelfeldman:trace-link-file

Conversation

@pavelfeldman
Copy link
Copy Markdown
Member

Summary

  • When opening a .trace file (not a .zip), create a .link file pointing to the original instead of extracting
  • loadTrace() checks for .link and loads from the linked location
  • Make traceLoader.load() accept optional traceFile filter and optional unzipProgress

if (fs.existsSync(linkFile)) {
const tracePath = await fs.promises.readFile(linkFile, 'utf-8');
traceDir = path.dirname(tracePath);
traceFile = path.basename(tracePath);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe support path to a dir too?

@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "MCP"

1 failed
❌ [msedge] › mcp/files.spec.ts:106 › clicking on download link emits download @mcp-windows-latest

6298 passed, 370 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "tests 1"

3 flaky ⚠️ [chromium-library] › library/browsertype-connect.spec.ts:682 › run-server › should record trace with sources `@chromium-ubuntu-22.04-node20`
⚠️ [chromium-library] › library/video.spec.ts:481 › screencast › should capture static page in persistent context @smoke `@chromium-ubuntu-22.04-node22`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1080 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-22.04-node20`

39111 passed, 846 skipped


Merge workflow run.

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.

3 participants