Skip to content

n3 + browser compatibility#43

Merged
giurgiur99 merged 3 commits into
mainfrom
browser-compatibility
Apr 15, 2026
Merged

n3 + browser compatibility#43
giurgiur99 merged 3 commits into
mainfrom
browser-compatibility

Conversation

@giurgiur99
Copy link
Copy Markdown
Contributor

@giurgiur99 giurgiur99 commented Apr 14, 2026

Fixes oceanprotocol/ocean.js#2063

Changes proposed in this PR:

Browser compatibility fix for SHACL validation.

  • Replace stream-based RDF parsing (@rdfjs/formats-common) with n3 + jsonld
  • Remove Node.js dependencies: crypto, fs, stream, path, url
  • Use ethers.sha256 instead of Node's crypto.createHash
  • Embed TTL schemas at build time via scripts/generate-schemas.js
  • Fix package.json: export -> exports, correct entry points

@giurgiur99 giurgiur99 marked this pull request as ready for review April 15, 2026 06:06
Copy link
Copy Markdown
Member

@alexcos20 alexcos20 left a comment

Choose a reason for hiding this comment

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

lgtm

@giurgiur99 giurgiur99 merged commit ebb2cba into main Apr 15, 2026
9 checks passed
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.

Remove nodejs deps in ocean.js (like fs used in ddo.js)

2 participants