Skip to content

Commit 04695d2

Browse files
BroccoBrocco
committed
refactor: Consolidate adding dependencies
1 parent 04893ca commit 04695d2

File tree

9 files changed

+438
-140
lines changed

9 files changed

+438
-140
lines changed

‎packages/schematics/angular/application/index.ts

Lines changed: 18 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ import {
2929
addProjectToWorkspace,
3030
getWorkspace,
3131
} from '../utility/config';
32+
import { NodeDependencyType, addPackageJsonDependency } from '../utility/dependencies';
3233
import { latestVersions } from '../utility/latest-versions';
3334
import { validateProjectName } from '../utility/validation';
3435
import { Schema as ApplicationOptions } from './schema';
@@ -60,29 +61,23 @@ import { Schema as ApplicationOptions } from './schema';
6061

6162
function addDependenciesToPackageJson() {
6263
return (host: Tree) => {
63-
const packageJsonPath = 'package.json';
64-
65-
if (!host.exists('package.json')) { return host; }
66-
67-
const source = host.read('package.json');
68-
if (!source) { return host; }
69-
70-
const sourceText = source.toString('utf-8');
71-
const json = JSON.parse(sourceText);
72-
73-
if (!json['devDependencies']) {
74-
json['devDependencies'] = {};
75-
}
76-
77-
json.devDependencies = {
78-
'@angular/compiler-cli': latestVersions.Angular,
79-
'@angular-devkit/build-angular': latestVersions.DevkitBuildAngular,
80-
'typescript': latestVersions.TypeScript,
81-
// De-structure last keeps existing user dependencies.
82-
...json.devDependencies,
83-
};
84-
85-
host.overwrite(packageJsonPath, JSON.stringify(json, null, 2));
64+
[
65+
{
66+
type: NodeDependencyType.Dev,
67+
name: '@angular/compiler-cli',
68+
version: latestVersions.Angular,
69+
},
70+
{
71+
type: NodeDependencyType.Dev,
72+
name: '@angular-devkit/build-angular',
73+
version: latestVersions.DevkitBuildAngular,
74+
},
75+
{
76+
type: NodeDependencyType.Dev,
77+
name: 'typescript',
78+
version: latestVersions.TypeScript,
79+
},
80+
].forEach(dependency => addPackageJsonDependency(host, dependency));
8681

8782
return host;
8883
};

‎packages/schematics/angular/library/index.ts

Lines changed: 42 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -27,23 +27,15 @@ import {
2727
addProjectToWorkspace,
2828
getWorkspace,
2929
} from '../utility/config';
30+
import {
31+
NodeDependencyType,
32+
addPackageJsonDependency,
33+
} from '../utility/dependencies';
3034
import { latestVersions } from '../utility/latest-versions';
3135
import { validateProjectName } from '../utility/validation';
3236
import { Schema as LibraryOptions } from './schema';
3337

3438

35-
type PackageJsonPartialType = {
36-
scripts: {
37-
[key: string]: string;
38-
},
39-
dependencies: {
40-
[key: string]: string;
41-
},
42-
devDependencies: {
43-
[key: string]: string;
44-
},
45-
};
46-
4739
interface UpdateJsonFn<T> {
4840
(obj: T): T | void;
4941
}
@@ -96,39 +88,45 @@ function updateTsConfig(packageName: string, distRoot: string) {
9688
function addDependenciesToPackageJson() {
9789

9890
return (host: Tree) => {
99-
if (!host.exists('package.json')) { return host; }
100-
101-
return updateJsonFile(host, 'package.json', (json: PackageJsonPartialType) => {
102-
103-
104-
if (!json['dependencies']) {
105-
json['dependencies'] = {};
106-
}
107-
108-
json.dependencies = {
109-
'@angular/common': latestVersions.Angular,
110-
'@angular/core': latestVersions.Angular,
111-
'@angular/compiler': latestVersions.Angular,
112-
// De-structure last keeps existing user dependencies.
113-
...json.dependencies,
114-
};
115-
116-
if (!json['devDependencies']) {
117-
json['devDependencies'] = {};
118-
}
91+
[
92+
{
93+
type: NodeDependencyType.Dev,
94+
name: '@angular/compiler-cli',
95+
version: latestVersions.Angular,
96+
},
97+
{
98+
type: NodeDependencyType.Dev,
99+
name: '@angular-devkit/build-ng-packagr',
100+
version: latestVersions.DevkitBuildNgPackagr,
101+
},
102+
{
103+
type: NodeDependencyType.Dev,
104+
name: '@angular-devkit/build-angular',
105+
version: latestVersions.DevkitBuildNgPackagr,
106+
},
107+
{
108+
type: NodeDependencyType.Dev,
109+
name: 'ng-packagr',
110+
version: '^3.0.0',
111+
},
112+
{
113+
type: NodeDependencyType.Dev,
114+
name: 'tsickle',
115+
version: '>=0.29.0',
116+
},
117+
{
118+
type: NodeDependencyType.Dev,
119+
name: 'tslib',
120+
version: '^1.9.0',
121+
},
122+
{
123+
type: NodeDependencyType.Dev,
124+
name: 'typescript',
125+
version: latestVersions.TypeScript,
126+
},
127+
].forEach(dependency => addPackageJsonDependency(host, dependency));
119128

120-
json.devDependencies = {
121-
'@angular/compiler-cli': latestVersions.Angular,
122-
'@angular-devkit/build-ng-packagr': latestVersions.DevkitBuildNgPackagr,
123-
'@angular-devkit/build-angular': latestVersions.DevkitBuildNgPackagr,
124-
'ng-packagr': '^3.0.0',
125-
'tsickle': '>=0.29.0',
126-
'tslib': '^1.9.0',
127-
'typescript': latestVersions.TypeScript,
128-
// De-structure last keeps existing user dependencies.
129-
...json.devDependencies,
130-
};
131-
});
129+
return host;
132130
};
133131
}
134132

‎packages/schematics/angular/migrations/update-6/index.ts

Lines changed: 14 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,16 @@ import {
2626
} from '@angular-devkit/schematics';
2727
import { NodePackageInstallTask } from '@angular-devkit/schematics/tasks';
2828
import { AppConfig, CliConfig } from '../../utility/config';
29-
import { latestVersions } from '../../utility/latest-versions';
3029
import {
31-
appendPropertyInAstObject,
30+
NodeDependency,
31+
NodeDependencyType,
32+
addPackageJsonDependency,
33+
} from '../../utility/dependencies';
34+
import {
3235
appendValueInAstArray,
3336
findPropertyInAstObject,
34-
} from './json-utils';
37+
} from '../../utility/json-utils';
38+
import { latestVersions } from '../../utility/latest-versions';
3539

3640
const defaults = {
3741
appRoot: 'src',
@@ -649,49 +653,13 @@ function updateSpecTsConfig(config: CliConfig): Rule {
649653

650654
function updatePackageJson(config: CliConfig) {
651655
return (host: Tree, context: SchematicContext) => {
652-
const pkgPath = '/package.json';
653-
const buffer = host.read(pkgPath);
654-
if (buffer == null) {
655-
throw new SchematicsException('Could not read package.json');
656-
}
657-
const pkgAst = parseJsonAst(buffer.toString(), JsonParseMode.Strict);
658-
659-
if (pkgAst.kind != 'object') {
660-
throw new SchematicsException('Error reading package.json');
661-
}
662-
663-
const devDependenciesNode = findPropertyInAstObject(pkgAst, 'devDependencies');
664-
if (devDependenciesNode && devDependenciesNode.kind != 'object') {
665-
throw new SchematicsException('Error reading package.json; devDependency is not an object.');
666-
}
667-
668-
const recorder = host.beginUpdate(pkgPath);
669-
const depName = '@angular-devkit/build-angular';
670-
if (!devDependenciesNode) {
671-
// Haven't found the devDependencies key, add it to the root of the package.json.
672-
appendPropertyInAstObject(recorder, pkgAst, 'devDependencies', {
673-
[depName]: latestVersions.DevkitBuildAngular,
674-
});
675-
} else {
676-
// Check if there's a build-angular key.
677-
const buildAngularNode = findPropertyInAstObject(devDependenciesNode, depName);
678-
679-
if (!buildAngularNode) {
680-
// No build-angular package, add it.
681-
appendPropertyInAstObject(
682-
recorder,
683-
devDependenciesNode,
684-
depName,
685-
latestVersions.DevkitBuildAngular,
686-
);
687-
} else {
688-
const { end, start } = buildAngularNode;
689-
recorder.remove(start.offset, end.offset - start.offset);
690-
recorder.insertRight(start.offset, JSON.stringify(latestVersions.DevkitBuildAngular));
691-
}
692-
}
693-
694-
host.commitUpdate(recorder);
656+
const dependency: NodeDependency = {
657+
type: NodeDependencyType.Dev,
658+
name: '@angular-devkit/build-angular',
659+
version: latestVersions.DevkitBuildAngular,
660+
overwrite: true,
661+
};
662+
addPackageJsonDependency(host, dependency);
695663

696664
context.addTask(new NodePackageInstallTask({
697665
packageManager: config.packageManager === 'default' ? undefined : config.packageManager,

‎packages/schematics/angular/service-worker/index.ts

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,10 @@ import {
2626
getWorkspace,
2727
getWorkspacePath,
2828
} from '../utility/config';
29+
import { addPackageJsonDependency, getPackageJsonDependency } from '../utility/dependencies';
2930
import { getAppModulePath } from '../utility/ng-ast-utils';
3031
import { Schema as ServiceWorkerOptions } from './schema';
3132

32-
const packageJsonPath = '/package.json';
33-
3433
function updateConfigFile(options: ServiceWorkerOptions): Rule {
3534
return (host: Tree, context: SchematicContext) => {
3635
context.logger.debug('updating config file.');
@@ -72,17 +71,15 @@ function addDependencies(): Rule {
7271
return (host: Tree, context: SchematicContext) => {
7372
const packageName = '@angular/service-worker';
7473
context.logger.debug(`adding dependency (${packageName})`);
75-
const buffer = host.read(packageJsonPath);
76-
if (buffer === null) {
77-
throw new SchematicsException('Could not find package.json');
74+
const coreDep = getPackageJsonDependency(host, '@angular/core');
75+
if (coreDep === null) {
76+
throw new SchematicsException('Could not find version.');
7877
}
79-
80-
const packageObject = JSON.parse(buffer.toString());
81-
82-
const ngCoreVersion = packageObject.dependencies['@angular/core'];
83-
packageObject.dependencies[packageName] = ngCoreVersion;
84-
85-
host.overwrite(packageJsonPath, JSON.stringify(packageObject, null, 2));
78+
const platformServerDep = {
79+
...coreDep,
80+
name: packageName,
81+
};
82+
addPackageJsonDependency(host, platformServerDep);
8683

8784
return host;
8885
};

‎packages/schematics/angular/universal/index.ts

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ import * as ts from 'typescript';
3434
import { findNode, getDecoratorMetadata } from '../utility/ast-utils';
3535
import { InsertChange } from '../utility/change';
3636
import { getWorkspace } from '../utility/config';
37+
import { addPackageJsonDependency, getPackageJsonDependency } from '../utility/dependencies';
3738
import { findBootstrapModuleCall, findBootstrapModulePath } from '../utility/ng-ast-utils';
3839
import { Schema as UniversalOptions } from './schema';
3940

@@ -172,18 +173,15 @@ function addServerTransition(options: UniversalOptions): Rule {
172173

173174
function addDependencies(): Rule {
174175
return (host: Tree) => {
175-
const pkgPath = '/package.json';
176-
const buffer = host.read(pkgPath);
177-
if (buffer === null) {
178-
throw new SchematicsException('Could not find package.json');
176+
const coreDep = getPackageJsonDependency(host, '@angular/core');
177+
if (coreDep === null) {
178+
throw new SchematicsException('Could not find version.');
179179
}
180-
181-
const pkg = JSON.parse(buffer.toString());
182-
183-
const ngCoreVersion = pkg.dependencies['@angular/core'];
184-
pkg.dependencies['@angular/platform-server'] = ngCoreVersion;
185-
186-
host.overwrite(pkgPath, JSON.stringify(pkg, null, 2));
180+
const platformServerDep = {
181+
...coreDep,
182+
name: '@angular/platform-server',
183+
};
184+
addPackageJsonDependency(host, platformServerDep);
187185

188186
return host;
189187
};

0 commit comments

Comments
 (0)