LinkedSoftwareDependencies / Components.js

🧩 A semantic dependency injection framework
https://componentsjs.readthedocs.io/
Other
41 stars 6 forks source link

Globally installed commands fail in presence of other modules #88

Closed RubenVerborgh closed 6 months ago

RubenVerborgh commented 2 years ago

Issue type:


Description:

Steps to reproduce:

npm install -g @comunica/query-sparql solid-chess
comunica-sparql-http https://www.rubensworks.net/

Note how comunica-sparql-http picks up the wrong Comunica 1.x version installed by solid-chess.

So on the one hand this is related to https://github.com/LinkedSoftwareDependencies/Components.js/issues/73, but on the other hand the problem is that too many components are being read in general (even if they had been 5.x).

Found by @Sindhu-Vasireddy


Environment:

Reproduced on Windows, macOS, Ubuntu

Crash log:

2022-03-20T14:33:38.512Z [Components.js] warn: Detected deprecated context URL 'https://linkedsoftwaredependencies.org/bundles/npm/componentsjs/^4.0.0/components/context.jsonld' in C:/Users/Administrator/AppData/Roaming/npm/node_modules/solid-chess/node_modules/@comunica/bus-rdf-source-identifier/components/Actor/RdfSourceIdentifier.jsonld. Prefer using version '^5.0.0' instead.
C:\Users\Administrator\AppData\Roaming\npm\node_modules\@comunica\query-sparql\node_modules\componentsjs\lib\loading\ComponentRegistry.js:95
            throw new Error(`Resource ${componentResource.value} is not a valid component, either it is not defined, has no type, or is incorrectly referenced${referencingComponent ? ` by ${referencingComponent.value}` : ''}.`);
                  ^

Error: Resource https://linkedsoftwaredependencies.org/bundles/npm/@comunica/bus-query-operation/Actor/QueryOperationTypedMediated is not a valid component, either it is not defined, has no type, or is incorrectly referenced by https://linkedsoftwaredependencies.org/bundles/npm/@comunica/actor-abstract-bindings-hash/Actor/AbstractBindingsHash.
    at ComponentRegistry.requireValidComponent (C:\Users\Administrator\AppData\Roaming\npm\node_modules\@comunica\query-sparql\node_modules\componentsjs\lib\loading\ComponentRegistry.js:95:19)
    at ComponentRegistryFinalizer.inheritParameters (C:\Users\Administrator\AppData\Roaming\npm\node_modules\@comunica\query-sparql\node_modules\componentsjs\lib\loading\ComponentRegistryFinalizer.js:51:36)
    at ComponentRegistryFinalizer.inheritParameters (C:\Users\Administrator\AppData\Roaming\npm\node_modules\@comunica\query-sparql\node_modules\componentsjs\lib\loading\ComponentRegistryFinalizer.js:57:18)
    at ComponentRegistryFinalizer.finalize (C:\Users\Administrator\AppData\Roaming\npm\node_modules\@comunica\query-sparql\node_modules\componentsjs\lib\loading\ComponentRegistryFinalizer.js:33:18)
    at ComponentsManagerBuilder.build (C:\Users\Administrator\AppData\Roaming\npm\node_modules\@comunica\query-sparql\node_modules\componentsjs\lib\loading\ComponentsManagerBuilder.js:86:28)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async instantiateComponent (C:\Users\Administrator\AppData\Roaming\npm\node_modules\@comunica\query-sparql\node_modules\@comunica\runner\lib\Setup.js:26:21)
    at async QueryEngineFactoryBase.create (C:\Users\Administrator\AppData\Roaming\npm\node_modules\@comunica\query-sparql\node_modules\@comunica\actor-init-query\lib\QueryEngineFactoryBase.js:34:24)
github-actions[bot] commented 2 years ago

Thanks for reporting!