e8fbfb14c1
Add coexistence checks to all enqueue methods to prevent loading both React and Grid.js assets simultaneously. Changes: - ReactAdmin.php: Only enqueue React assets when ?react=1 - Init.php: Skip Grid.js when React active on admin pages - Form.php, Coupon.php, Access.php: Restore classic assets when ?react=0 - Customer.php, Product.php, License.php: Add coexistence checks Now the toggle between Classic and React versions works correctly. Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
89 lines
2.1 KiB
JavaScript
89 lines
2.1 KiB
JavaScript
/** @type {import('eslint').Rule.RuleModule} */
|
|
module.exports = {
|
|
type: 'problem',
|
|
meta: {
|
|
schema: [
|
|
{
|
|
type: 'object',
|
|
additionalProperties: false,
|
|
patternProperties: {
|
|
'^@wordpress\\/[a-zA-Z0-9_-]+$': {
|
|
type: 'array',
|
|
uniqueItems: true,
|
|
minItems: 1,
|
|
items: {
|
|
type: 'string',
|
|
pattern: '^(?:__experimental|__unstable)',
|
|
},
|
|
},
|
|
},
|
|
},
|
|
],
|
|
},
|
|
create( context ) {
|
|
/** @type {AllowedImportsMap} */
|
|
const allowedImports =
|
|
( context.options &&
|
|
typeof context.options[ 0 ] === 'object' &&
|
|
context.options[ 0 ] ) ||
|
|
{};
|
|
const reporter = makeListener( { allowedImports, context } );
|
|
|
|
return { ImportDeclaration: reporter };
|
|
},
|
|
};
|
|
|
|
/**
|
|
* @param {Object} _
|
|
* @param {AllowedImportsMap} _.allowedImports
|
|
* @param {import('eslint').Rule.RuleContext} _.context
|
|
*
|
|
* @return {(node: Node) => void} Listener function
|
|
*/
|
|
function makeListener( { allowedImports, context } ) {
|
|
return function reporter( node ) {
|
|
if ( node.type !== 'ImportDeclaration' ) {
|
|
return;
|
|
}
|
|
if ( typeof node.source.value !== 'string' ) {
|
|
return;
|
|
}
|
|
|
|
const sourceModule = node.source.value.trim();
|
|
|
|
// Ignore non-WordPress packages.
|
|
if ( ! sourceModule.startsWith( '@wordpress/' ) ) {
|
|
return;
|
|
}
|
|
|
|
const allowedImportNames = allowedImports[ sourceModule ] || [];
|
|
|
|
node.specifiers.forEach( ( specifierNode ) => {
|
|
if ( specifierNode.type !== 'ImportSpecifier' ) {
|
|
return;
|
|
}
|
|
|
|
const importedName = specifierNode.imported.name;
|
|
|
|
if (
|
|
! importedName.startsWith( '__unstable' ) &&
|
|
! importedName.startsWith( '__experimental' )
|
|
) {
|
|
return;
|
|
}
|
|
|
|
if ( allowedImportNames.includes( importedName ) ) {
|
|
return;
|
|
}
|
|
|
|
context.report( {
|
|
message: `Usage of \`${ importedName }\` from \`${ sourceModule }\` is not allowed.\nSee https://developer.wordpress.org/block-editor/contributors/develop/coding-guidelines/#experimental-and-unstable-apis for details.`,
|
|
node: specifierNode,
|
|
} );
|
|
} );
|
|
};
|
|
}
|
|
|
|
/** @typedef {import('estree').Node} Node */
|
|
/** @typedef {Record<string, string[]|undefined>} AllowedImportsMap */
|