Fix dynamic absolute path import for Windows fix#477

[aolyang]

this PR fix issue #477

according to nodejs/node#31710 and https://nodejs.org/api/url.html#url_url_pathtofileurl_path

import { pathToFileURL } from 'node:url';

new URL('/foo#1', 'file:');           // Incorrect: file:///foo#1
pathToFileURL('/foo#1');              // Correct:   file:///foo%231 (POSIX)

new URL('/some/path%.c', 'file:');    // Incorrect: file:///some/path%.c
pathToFileURL('/some/path%.c');       // Correct:   file:///some/path%25.c (POSIX)

steps

0. test the fixture is right by npm patch package.
1. fork and install, follow CONTRIBUTING.md
2. change next-contentlyer-example/contentlayer.config.ts add onSuccess hook

   export default makeSource({
     contentDirPath: 'posts',
     documentTypes: [Post],
   +  onSuccess: async (importData) => {
   +    const { allDocuments } = await importData()
   +    console.log('Contentlayer data successfully generated.', allDocuments.length)
   +  }
   })

3. build next-contentlyer-example with no change, error occur.
   
4. update packages\@contentlayer\core\src\generation\generate-dotpkg.ts, yarn build at root dir.
5. rebuild success;
6. test again on Debian linux.

win bash

debian

[changeset-bot]

⚠️ No Changeset found

Latest commit: 35ee674

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR

Some errors occurred when validating the changesets config:
The package or glob expression "examples-*" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

[PxlSyl]

Hi, same error here. Will it be fixed soon?

[aolyang]

@PxlSyl follow this issue can help you timlrx/tailwind-nextjs-starter-blog#704

Im not sure when this PR will be accepted.

[AltairAIDev]

Accept pls

[stale]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[rreo]

Why was this closed?

[timlrx]

@aolyang I have merged the fix in timlrx/contentlayer2#7. Feel free to try out v0.4.5 of contentlayer2 and let me know if it now works well in Windows, thanks!

[aolyang]

git clone https://github.com/aolyang/blogs.git contentlayer2-test --depth=1 # currently contentlayer version
cd contentlayer2-test
pnpm i
pnpm dev # it's fixed by patched, works

Then pnpm patch-remove @contentlayer/core@0.3.4 undo the fix.

Don't forget remove .contentlayer directory (caches), and errors back.

pnpm rm contentlayer next-contentlayer -r
pnpm add contentlayer@npm:contentlayer2 `
>          next-contentlayer@npm:next-contentlayer2 `
>          -D
pnpm dev

URL error is gone (if I not use mdx frontmatter), but another problem appear, stucked by timlrx/contentlayer2#15

I will try it latter if the problem fixed.

Say somthing complaint, dont be mind.
Currently, self package-pathces is good to go.
This code base use effect-ts compose logic, and so many short name alias.
Codes and Effect-ts is good if learn it through standlone function, but block me maybe others too from mid journey of this repo.
And not that active. (maybe) 🤣
And not that willing to accept PR.
Maybe abondon this ship, try Nextra or Astro is another good chice.