Skip to content

Conversation

Copy link

Copilot AI commented Dec 19, 2025

Disable Sourcemap Generation ✅ COMPLETED

Changes made:

  • Remove "sourceMap": true from tsconfig.json to disable sourcemap generation
  • Revert package-lock.json changes (per maintainer request)
  • Verify no .js.map files are generated
  • Run tests to ensure functionality is intact

Summary:
This PR disables sourcemap generation to resolve Vitest warnings about missing source files. Libraries don't need sourcemaps in distribution since they're not useful for end users.

Verification:

  • ✅ Build successful with no .map files generated
  • ✅ All 84 tests pass (82 passed, 1 skipped, 1 todo)
  • ✅ No sourcemap references in JavaScript files
  • ✅ Package-lock.json reverted to original state
Original prompt

This section details on the original issue you should resolve

<issue_title>Sourcemap references missing source files</issue_title>
<issue_description>When using Vitest, I get warnings like:

Sourcemap for "mypath/node_modules/.pnpm/superjson@2.2.6/node_modules/superjson/dist/index.js" points to missing source files

The published sourcemap references source files that aren't included in the package.

Suggested fix: Enable inlineSources in tsconfig.json to embed the source content directly in the .map files:

{
  "compilerOptions": {
    "sourceMap": true,
    "inlineSources": true
  }
}

This resolves the warnings without adding extra files to the package.</issue_description>

Comments on the Issue (you are @copilot in this section)

@Skn0tt Hmm, why are we emitting sourcemaps in the first place? No need for that in a library like ours. @copilot please disable sourcemaps.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits December 19, 2025 13:40
Co-authored-by: Skn0tt <14912729+Skn0tt@users.noreply.github.com>
Co-authored-by: Skn0tt <14912729+Skn0tt@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix sourcemap references to include source files Disable sourcemap generation to eliminate missing source file warnings Dec 19, 2025
Copilot AI requested a review from Skn0tt December 19, 2025 13:53
Copy link
Collaborator

@Skn0tt Skn0tt left a comment

Choose a reason for hiding this comment

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

@copilot please revert the package lock changes

Co-authored-by: Skn0tt <14912729+Skn0tt@users.noreply.github.com>
Copy link
Author

Copilot AI commented Dec 19, 2025

@copilot please revert the package lock changes

Reverted in 4354735.

Copy link
Collaborator

@Skn0tt Skn0tt left a comment

Choose a reason for hiding this comment

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

@copilot please revert the package lock changes

Copilot AI requested a review from Skn0tt December 19, 2025 16:02
@Skn0tt Skn0tt marked this pull request as ready for review December 19, 2025 16:02
@Skn0tt Skn0tt requested a review from flybayer December 19, 2025 16:02
@Skn0tt Skn0tt self-requested a review December 19, 2025 16:44
@Skn0tt
Copy link
Collaborator

Skn0tt commented Dec 19, 2025

My review is required for merging, but it doesn't count my approval because Copilot authored on my behalf 😢

Screenshot 2025-12-19 at 17 44 27

@Skn0tt Skn0tt enabled auto-merge (squash) December 23, 2025 10:05
@baraknaveh
Copy link

Any task remaining before this could be merged?

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.

Sourcemap references missing source files

4 participants