GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Esm by bourgeoa · Pull Request #1834 · nodeSolidServer/node-solid-server

Conversation

@bourgeoa

@bourgeoa bourgeoa commented Dec 23, 2025

Copy link
Copy Markdown
Member

No description provided.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
bourgeoa and others added 30 commits November 13, 2025 17:37
@bourgeoa
mixed ESM
6f6e708
@bourgeoa
more mixed for app
6e0b966
@bourgeoa
more mixed for app and test
78c0558
@bourgeoa
index.mjs
f3f3968
CI Fix
Fix line endings in bin/solid for Docker compatibility
d5d0e75
- Convert Windows CRLF line endings to Unix LF in bin/solid script
- Add .gitattributes to enforce proper line endings for executable files
- Fixes Docker CI error: 'node\r': No such file or directory
CI Fix
package.json
13e5054
CI Fix
.gitattributes
0357fea
CI Fix
test/ migrated to esm in test-esm/
5b65f43
CI Fix
ESM Migration: Convert main lib files and lib/api to ESM
ac4eec7
- Create ESM versions (.mjs) for 7 main lib files:
  - capability-discovery.mjs
  - ldp-container.mjs
  - ldp.mjs (large 632-line file)
  - payment-pointer-discovery.mjs
  - rdf-notification-template.mjs
  - resource-mapper.mjs (227 lines)
  - server-config.mjs
- Convert lib/api/ directory to ESM:
  - index.mjs
  - accounts/user-accounts.mjs
  - authn/index.mjs
  - authn/force-user.mjs
- All files maintain dual CommonJS/ESM compatibility
- Proper import/export conversions with createRequire for CommonJS deps
- No errors detected in new ESM modules
- Total: 18 .mjs files now available in lib/ directory
CI Fix
ESM Migration: Complete lib/api/ and lib/handlers/ conversion
73b2626
lib/api/ conversion completed:
- authn/webid-tls.mjs (70 lines - TLS authentication)
- authn/webid-oidc.mjs (203 lines - OIDC authentication)

lib/handlers/ conversion completed (7 new .mjs files):
- restrict-to-top-domain.mjs (simple middleware)
- options.mjs (HTTP OPTIONS handler)
- index.mjs (directory index handler)
- auth-proxy.mjs (authentication proxy)
- cors-proxy.mjs (96 lines - CORS proxy with IP filtering)
- notify.mjs (149 lines - notification system)
- error-pages.mjs (213 lines - error handling & databrowser)

Progress:
- lib/ main files:  9/9 converted
- lib/api/:  6/6 converted
- lib/handlers/:  14/14 converted
- All new modules error-free and maintain dual compatibility
@bourgeoa
Add ESM test command to CI workflow
277633e
CI Fix
add missing test-esm/resources
562a91b
CI Fix
update esm and tests
d2ebe52
CI Fix
add missing .mjs files
86da28f
CI Fix
missing mjs file
04fef95
CI Fix
add esm entry point
f3d0a1b
CI Fix
missing content in ldp.mjs
60e2f93
CI Fix
miscellaneous updates
53c8278
CI Fix
updates lib/api
635f7b2
CI Fix
updates to lib/srvices
ebf457f
CI Fix
updates to lib/models
7bd6cdd
CI Fix
updates to lib/payment-pointer-discovery.mjs
aba64d9
CI Fix
updates to lib/ldp-container.mjs
b935bdf
CI Fix
updates to lib/requests/auth-request.mjs
3f536ab
CI Fix
updates to lib/rdf-notification-template.mjs
beb6dad
CI Fix
updates to lib/handlers
c668ad3
CI Fix
lib/server-config.mjs
5c258b2
CI Fix
updates .mjs
b370ff7
CI Fix
Revert "updates .mjs"
1bf6b36
This reverts commit b370ff7.
CI Fix
fix critical node-forge
fd1a323
bourgeoa added 30 commits December 8, 2025 22:31
@bourgeoa
Merge branch 'main' into nss2esm-hybrid
924d4b8
@bourgeoa
update require() and debug's
1e3025d
@bourgeoa
delete patch.mjs.old
3f1fba3
@bourgeoa
updates
6db0f28
@bourgeoa
resources/config
7610c97
@bourgeoa
rename test-esm to test
23f2403
@bourgeoa
standard issues
94454d1
@bourgeoa
remove test-esm from workflow
951d712
@bourgeoa
rm *.js to keep *.mjs
9424280
@bourgeoa
update bin/solid and import index.mjs
e1b12bb
@bourgeoa
c8 in CI workflow
3544e9b
@bourgeoa
add missing test files
40da84a
@bourgeoa
some cleaning
c65a248
@bourgeoa
mashlib was not loading and server OIDC listening issue
bb700b5
@bourgeoa
npm publish
f1cbcc9
@bourgeoa
Change test suite runner to ubuntu-latest
4d031e4
@bourgeoa
Refactor CI workflow by removing test-suite job
822b18b
Removed the test-suite job and updated dependencies for npm-publish-build and npm-publish-latest.
@bourgeoa
oidc to include ES256/384/512
9436591
@bourgeoa
update patch tests to [email protected]
954a10d
@bourgeoa
Merge branch 'esm' of https://github.com/nodeSolidServer/node-solid-s…
c012117
…erver into esm
@bourgeoa
replace node-fetch with node:fetch and update for isIp and some lints
f7d9560
@bourgeoa
remove standard and use eslint@9
ad1867f
@bourgeoa
npm run lint in CI/CD
f18e73e
@bourgeoa
update more dependencies
bf760f9
@bourgeoa
eslint.config.mjs
1185df4
@bourgeoa
rimraf and get-folder-size at @latest
b387f8c
@bourgeoa
http-proxy-middleware@3
c7c332a
@bourgeoa
http-proxy-middleware comments
b278bff
@bourgeoa
rfc9207 iss == idp
7eb7ffb
@bourgeoa
package-lock.json security issue
95a4630
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@bourgeoa
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.