Skip to content

Commit 6b54dd8

Browse files
authored
Fix internal source map warnings (#19788)
This PR fixes an internal source map warning when running tests. To solve this, we encode the `=` as `\x3d` instead. I'm not 100% sure if Vitest was hanging on this but it solved the following warning: ``` ✓ @tailwindcss/cli src/utils/format-ns.test.ts (21 tests) 3ms 11:43:59 AM [vite] (ssr) Failed to load source map for /Users/robin/github.com/tailwindlabs/tailwindcss/packages/@tailwindcss-node/dist/index.mjs. Error: An error occurred while trying to read the map file at ${i} Error: ENOENT: no such file or directory, open '/Users/robin/github.com/tailwindlabs/tailwindcss/packages/@tailwindcss-node/dist/${i}' at open (node:internal/fs/promises:634:25) at Object.readFile (node:internal/fs/promises:1238:14) at extractSourcemapFromFile (file:///Users/robin/github.com/tailwindlabs/tailwindcss/node_modules/.pnpm/vite@7.0.0_@types+node@20.19.1_jiti@2.6.1_lightningcss@1.31.1_patch_hash=tzyxy3asfxcqc7ihroou_7epcep7uhfc7zidzdmxzgvdzwi/node_modules/vite/dist/node/chunks/dep-Bsx9IwL8.js:8349:65) at loadAndTransform (file:///Users/robin/github.com/tailwindlabs/tailwindcss/node_modules/.pnpm/vite@7.0.0_@types+node@20.19.1_jiti@2.6.1_lightningcss@1.31.1_patch_hash=tzyxy3asfxcqc7ihroou_7epcep7uhfc7zidzdmxzgvdzwi/node_modules/vite/dist/node/chunks/dep-Bsx9IwL8.js:26405:22) ``` ## Test plan 1. Existing tests pass 2. Added a failing test to ensure that the source maps are emitted correctly
1 parent ad9fdef commit 6b54dd8

2 files changed

Lines changed: 9 additions & 1 deletion

File tree

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
import { expect, it } from 'vitest'
2+
import { toSourceMap } from './source-maps'
3+
4+
it('should emit source maps', () => {
5+
let map = toSourceMap('{"version":3,"sources":[],"names":[],"mappings":""}')
6+
7+
expect(map.comment('app.css.map')).toBe('/*# sourceMappingURL=app.css.map */\n')
8+
})

packages/@tailwindcss-node/src/source-maps.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ export function toSourceMap(map: DecodedSourceMap | string): SourceMap {
4646
let raw = typeof map === 'string' ? map : serializeSourceMap(map)
4747

4848
function comment(url: string) {
49-
return `/*# sourceMappingURL=${url} */\n`
49+
return `/*# sourceMappingURL\x3d${url} */\n`
5050
}
5151

5252
return {

0 commit comments

Comments
 (0)