Skip to content

WIP: Feat: Add support for shex imports to @ldo/cli#143

Draft
mrkvon wants to merge 6 commits intoo-development:mainfrom
mrkvon:feat-33-import-shex
Draft

WIP: Feat: Add support for shex imports to @ldo/cli#143
mrkvon wants to merge 6 commits intoo-development:mainfrom
mrkvon:feat-33-import-shex

Conversation

@mrkvon
Copy link
Copy Markdown
Contributor

@mrkvon mrkvon commented Mar 13, 2026

Work in progress.

  • import shex from other file
  • nested dependencies
  • circular dependencies
  • import from web
  • clean up and refactor

Issues to address:

  • do not use absolute IRIs (paths) for local files. That's not transferrable to other filesystems and leaks information about person's filesystem.
  • typings should be imported as import type, but dts-dom doesn't seem to support that. Maybe we can run linter to fix this (fixable issue).
  • cosmetic: import { Type as Type } from "./somewhere" could become import { Type } when they match.
  • don't export from shapeType what's not defined in the shape itself.
  • contexts need to be merged in some way. We need to be careful about term overlap (like schema:name vs foaf:name etc.)
    • Best would be nested contexts, but LDO dataset-proxy's nested context only seems to work 1 level deep
    • Alternative would be to make sure names are deduplicated across the import graph.
  • Currently only imports where imported schema appears as object within other schema. (Only that is tested). Generalize?

@mrkvon mrkvon force-pushed the feat-33-import-shex branch from 4234753 to 6bbdc0e Compare March 13, 2026 12:25
@mrkvon mrkvon force-pushed the feat-33-import-shex branch 2 times, most recently from d3326da to a404488 Compare March 26, 2026 16:05
mrkvon added 4 commits March 28, 2026 00:35
Putting things to what feels like more appropriate places,
but it's not perfect.
for example import type

TODO make sure .eslintrc is imported
@mrkvon mrkvon force-pushed the feat-33-import-shex branch from 255fb3b to 6f224c0 Compare March 28, 2026 00:23
@mrkvon mrkvon force-pushed the feat-33-import-shex branch from 6f224c0 to f53c09d Compare March 28, 2026 00:38
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.

1 participant