Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/azure-theme",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/houdini-utils",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/pierce-dom",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/react-chat",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/react-data-grid-react-window",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/react-data-grid-react-window-grid",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/react-draggable-dialog",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/react-gamepad-navigation",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/react-interactive-tab",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/react-keytips",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/react-resize-handle",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/react-themeless-provider",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/react-tree-grid",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/stylelint-plugin",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/teams-components",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "fix: normalize package dependencies",
"packageName": "@fluentui-contrib/variant-theme",
"email": "hochelmartin@gmail.com",
"dependentChangeType": "patch"
}
37 changes: 20 additions & 17 deletions eslint.config.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
const prettierConfig = require('eslint-config-prettier');
const nxEslintPlugin = require('@nx/eslint-plugin');
const eslintPluginImport = require('eslint-plugin-import');
const rnxKitEslintPlugin = require('@rnx-kit/eslint-plugin');
const stylisticPlugin = require('@stylistic/eslint-plugin');
const jsonParser = require('jsonc-eslint-parser');
Expand All @@ -26,7 +25,6 @@ module.exports = [
{
plugins: {
'@nx': nxEslintPlugin,
import: eslintPluginImport,
'@rnx-kit': rnxKitEslintPlugin,
'@stylistic': stylisticPlugin,
},
Expand Down Expand Up @@ -67,20 +65,6 @@ module.exports = [
'@rnx-kit/no-export-all': 'error',
},
},
{
files: ['**/*.ts?(x)', '**/*.js?(x)'],
rules: { 'import/no-extraneous-dependencies': ['error'] },
ignores: [
'**/*.test.[jt]s?(x)',
'**/*.spec.[jt]s?(x)',
'**/*.stories.[jt]s?(x)',
'**/*.component-browser-spec.tsx',
'**/generators/**/files/**',
'**/.storybook/**',
'**/playwright.config.ts',
'**/playwright/**',
],
},
{
files: ['**/*.spec.[jt]s?(x)', '**/*.test.[jt]s?(x)'],
languageOptions: {
Expand Down Expand Up @@ -136,6 +120,25 @@ module.exports = [
languageOptions: {
parser: jsonParser,
},
rules: {},
rules: {
'@nx/dependency-checks': [
'error',
{
includeTransitiveDependencies: false,
ignoredDependencies: [
'@types/react',
'@types/react-dom',
'@swc/helpers',
],
ignoredFiles: [
'{projectRoot}/playwright.config.ts',
'{projectRoot}/playwright/**',
'{projectRoot}/.storybook/**',
'{projectRoot}/stories/**',
'{projectRoot}/**/*.component-browser-spec.tsx',
],
},
],
},
},
];
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,6 @@
"beachball": "^2.33.2",
"eslint": "9.21.0",
"eslint-config-prettier": "10.0.2",
"eslint-plugin-import": "2.31.0",
"eslint-plugin-playwright": "2.2.0",
"globals": "15.15.0",
"jest": "29.7.0",
Expand Down
8 changes: 1 addition & 7 deletions packages/azure-theme/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
"name": "@fluentui-contrib/azure-theme",
"version": "0.1.0",
"dependencies": {
"@fluentui/react-icons": ">=2.0.204 <3.0.0",
"@swc/helpers": "~0.5.2"
},
"main": "./src/index.js",
Expand All @@ -14,11 +13,6 @@
]
},
"peerDependencies": {
"@fluentui/react-components": ">=9.46.3 <10.0.0",
"@fluentui/react-icons": ">=2.0.204 <3.0.0",
"@types/react": ">=16.8.0 <19.0.0",
"@types/react-dom": ">=16.8.0 <19.0.0",
"react": ">=16.8.0 <19.0.0",
"react-dom": ">=16.8.0 <19.0.0"
"@fluentui/react-components": ">=9.46.3 <10.0.0"
}
}
3 changes: 2 additions & 1 deletion packages/azure-theme/project.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@
"options": {
"lintFilePatterns": [
"packages/azure-theme/**/*.ts",
"packages/azure-theme/**/*.tsx"
"packages/azure-theme/**/*.tsx",
"packages/azure-theme/package.json"
]
}
},
Expand Down
9 changes: 1 addition & 8 deletions packages/houdini-utils/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,5 @@
"@swc/helpers": "~0.5.2"
},
"main": "./src/index.js",
"typings": "./src/index.d.ts",
"peerDependencies": {
"@fluentui/react-components": ">=9.46.3 <10.0.0",
"@types/react": ">=16.8.0 <19.0.0",
"@types/react-dom": ">=16.8.0 <19.0.0",
"react": ">=16.8.0 <19.0.0",
"react-dom": ">=16.8.0 <19.0.0"
}
"typings": "./src/index.d.ts"
}
3 changes: 2 additions & 1 deletion packages/houdini-utils/project.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@
"options": {
"lintFilePatterns": [
"packages/houdini-utils/**/*.ts",
"packages/houdini-utils/**/*.tsx"
"packages/houdini-utils/**/*.tsx",
"packages/houdini-utils/package.json"
]
}
},
Expand Down
3 changes: 1 addition & 2 deletions packages/nx-plugin/project.json
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,7 @@
"packages/nx-plugin/**/*.ts",
"packages/nx-plugin/package.json",
"packages/nx-plugin/generators.json",
"packages/nx-plugin/executors.json",
"packages/nx-plugin/package.json"
"packages/nx-plugin/executors.json"
]
}
},
Expand Down
29 changes: 28 additions & 1 deletion packages/nx-plugin/src/generators/component/generator.spec.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
import { createTreeWithEmptyWorkspace } from '@nx/devkit/testing';
import { Tree, joinPathFragments, readProjectConfiguration } from '@nx/devkit';
import {
Tree,
joinPathFragments,
readJson,
readProjectConfiguration,
workspaceRoot,
} from '@nx/devkit';

import libraryGenerator from '../library/generator';
import {
Expand All @@ -9,6 +15,7 @@ import {

import generator from './generator';
import { ComponentGeneratorSchema } from './schema';
import { getPackagePaths } from '../../utils';

describe('component generator', () => {
let tree: Tree;
Expand Down Expand Up @@ -68,4 +75,24 @@ describe('component generator', () => {
"
`);
});

describe(`package.json`, () => {
it('should update peer dependencies', async () => {
const config = readProjectConfiguration(tree, options.name);
const paths = getPackagePaths(workspaceRoot, config.root);
let pkgJson = readJson(tree, paths.packageJson);

expect(pkgJson.peerDependencies).toEqual(undefined);

await generator(tree, options);

pkgJson = readJson(tree, paths.packageJson);

expect(pkgJson.peerDependencies).toEqual({
'@fluentui/react-components': '>=9.46.3 <10.0.0',
'@types/react': '>=16.8.0 <19.0.0',
react: '>=16.8.0 <19.0.0',
});
});
});
});
4 changes: 4 additions & 0 deletions packages/nx-plugin/src/generators/component/generator.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import { formatFiles, generateFiles, Tree, workspaceRoot } from '@nx/devkit';
import * as path from 'path';
import { ComponentGeneratorSchema } from './schema';
import { getPackagePaths, getProject, npmScope } from '../../utils';
import { updateDependencies } from './update-dependencies';

export default async function (tree: Tree, options: ComponentGeneratorSchema) {
const { name, componentName } = options;
Expand All @@ -25,5 +26,8 @@ export default async function (tree: Tree, options: ComponentGeneratorSchema) {
'\n' +
main;
tree.write(paths.main, main);

updateDependencies(tree, { paths });

await formatFiles(tree);
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
import { readJson, updateJson, type Tree } from '@nx/devkit';
import { getPackagePaths, type PackageJson } from '../../utils';

function getReactComponentsVersion(tree: Tree): string {
const monorepoRootPkgJson: PackageJson = readJson(tree, '/package.json');
const { dependencies = {}, devDependencies = {} } = monorepoRootPkgJson;
const reactComponentsVersion =
dependencies['@fluentui/react-components'] ||
devDependencies['@fluentui/react-components'];

if (!reactComponentsVersion) {
throw new Error(
'🚨 Could not find @fluentui/react-components in package.json. please report this issue'
);
}

// strip version ranges non-numeric characters
return reactComponentsVersion.replace(/^[~^]/, '');
}

export function updateDependencies(
tree: Tree,
options: { paths: ReturnType<typeof getPackagePaths> }
) {
const reactComponentsVersion = getReactComponentsVersion(tree);
const peerDeps = {
'@fluentui/react-components': `>=${reactComponentsVersion} <10.0.0`,
'@types/react': '>=16.8.0 <19.0.0',
react: '>=16.8.0 <19.0.0',
};

updateJson(tree, options.paths.packageJson, (packageJson) => {
packageJson.peerDependencies = packageJson.peerDependencies ?? {};

Object.assign(packageJson.peerDependencies, peerDeps);

return packageJson;
});

return tree;
}
Loading