getThemePath() fails when a generated package is located in a mono-repo

[pomek]

📝 Provide detailed reproduction steps (if any)

1. Create a mono-repo (and use workspaces).
2. Inside the mono-repo, generate a new package.
3. Then, install dependencies.
4. Then, execute yarn start

✔️ Expected result

It does not throw any errors.

❌ Actual result

$ ckeditor5-package-tools start
node:internal/modules/cjs/loader:998
  throw err;
  ^

Error: Cannot find module '/Users/pomek/Projects/.../packages/ckeditor5-foo/node_modules/@ckeditor/ckeditor5-theme-lark/package.json'

❓ Possible solution

• It tries to load the package.json file from node_modules/ inside ckeditor5-foo.
• Replacing these lines

  ckeditor5-package-generator/packages/ckeditor5-package-tools/lib/utils/get-theme-path.js

  Lines 22 to 23 in 8f40903

  	const packagePath = path.join( cwd, 'node_modules', '@ckeditor', 'ckeditor5-theme-lark' );
  	const packageJson = require( path.join( packagePath, 'package.json' ) );

  with the following code helps:

const packagePath = require.resolve( '@ckeditor/ckeditor5-theme-lark/package.json' );
const packageJson = require( packagePath );

[pomek]

The same applies to

ckeditor5-package-generator/packages/ckeditor5-package-tools/lib/utils/get-webpack-config-dll.js

Lines 30 to 31 in 8f40903

	// An absolute path to the manifest file that the `DllReferencePlugin` plugin uses for mapping dependencies.
	const ckeditor5manifestPath = path.join( options.cwd, 'node_modules', 'ckeditor5', 'build', 'ckeditor5-dll.manifest.json' );

Fix:

// An absolute path to the manifest file that the `DllReferencePlugin` plugin uses for mapping dependencies.
const ckeditor5manifestPath = require.resolve( 'ckeditor5/build/ckeditor5-dll.manifest.json' );

[klonwar]

TLDR:
Can be fixed using yarn patch and changing getThemePath function from lib/utils:

const packageJsonPath = require.resolve( '@ckeditor/ckeditor5-theme-lark/package.json');
const packageJson = require(packageJsonPath);
const packagePath = path.join(packageJsonPath, '..');

There are two workarounds:

1. The first is to disable hoisting for the package:

"packages/ckeditor5-whatever/package.json

"installConfig": {
    "hoistingLimits": "dependencies"
}

But this is very bad as dependencies are duplicated during installation

2. The second is to use yarn patch

Tried to solve this issue using yarn patch command and applying your fixes. It really works. but still somethig is wrong:

Seems like some theme deps still not loaded correctly

UPD:
The reason is that original path.join( cwd, 'node_modules', '@ckeditor', 'ckeditor5-theme-lark' ); results in package dir path, while require.resolve( '@ckeditor/ckeditor5-theme-lark/package.json' ); results in package.json path.

The correct fix will be

module.exports = function getThemePath( cwd ) {
  const packageJsonPath = require.resolve( '@ckeditor/ckeditor5-theme-lark/package.json');
  const packageJson = require(packageJsonPath);
  const packagePath = path.join(packageJsonPath, '..');

  return path.join( packagePath, packageJson.main );
};

P.S. Trying to resolve require.resolve( '@ckeditor/ckeditor5-theme-lark'); gives .../node_modules/@ckeditor/ckeditor5-theme-lark/theme/theme.css so this is not an option

Hope this will be in release someday