Skip to content

Commit

Permalink
add JavaDependencyVersion and addJavaDefinition abstraction
Browse files Browse the repository at this point in the history
  • Loading branch information
mshima committed Mar 18, 2024
1 parent fa80e56 commit 790909c
Show file tree
Hide file tree
Showing 10 changed files with 75 additions and 40 deletions.
7 changes: 4 additions & 3 deletions generators/gradle/generator.ts
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ import {
addGradlePluginManagementCallback,
addGradlePropertyCallback,
addGradleBuildSrcDependencyCallback,
addGradleDependencyCatalogVersionCallback,
addGradleDependenciesCatalogVersionCallback,
addGradleBuildSrcDependencyCatalogVersionCallback,
addGradleDependencyCatalogLibrariesCallback,
addGradleDependencyCatalogPluginsCallback,
Expand Down Expand Up @@ -88,8 +88,9 @@ export default class GradleGenerator extends BaseApplicationGenerator {
source.addGradleProperty = property => this.editFile('gradle.properties', addGradlePropertyCallback(property));
source.addGradleBuildSrcDependency = dependency =>
this.editFile(`${GRADLE_BUILD_SRC_DIR}/build.gradle`, addGradleBuildSrcDependencyCallback(dependency));
source.addGradleDependencyCatalogVersion = version =>
this.editFile('gradle/libs.versions.toml', addGradleDependencyCatalogVersionCallback(version));
source.addGradleDependencyCatalogVersions = versions =>
this.editFile('gradle/libs.versions.toml', addGradleDependenciesCatalogVersionCallback(versions));
source.addGradleDependencyCatalogVersion = version => source.addGradleDependencyCatalogVersions!([version]);
source.addGradleDependencyCatalogLibraries = (libs, { gradleFile = 'build.gradle' } = {}) => {
libs = [...libs].sort((a, b) => a.libraryName.localeCompare(b.libraryName));
this.editFile('gradle/libs.versions.toml', addGradleDependencyCatalogLibrariesCallback(libs));
Expand Down
6 changes: 3 additions & 3 deletions generators/gradle/internal/needles.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ import type {
GradleTomlPlugin,
} from '../types.js';

const tomlItemToString = (item: Record<string, string>) =>
const tomlItemToString = (item: Record<string, string | undefined>) =>
`{ ${Object.entries(item)
.filter(([_key, value]) => value !== undefined)
.map(([key, value]) => `${key} = "${value}"`)
Expand Down Expand Up @@ -84,10 +84,10 @@ export const addGradleBuildSrcDependencyCallback = ({ groupId, artifactId, versi
contentToAdd: `${scope} "${groupId}:${artifactId}${version ? `:${version}` : ''}"`,
});

export const addGradleDependencyCatalogVersionCallback = ({ name, version }: GradleTomlVersion) =>
export const addGradleDependenciesCatalogVersionCallback = (versions: GradleTomlVersion[]) =>
createNeedleCallback({
needle: 'gradle-dependency-catalog-version',
contentToAdd: `${name} = "${version}"`,
contentToAdd: versions.map(({ name, version }) => `${name} = "${version}"`),
});

export const addGradleDependencyCatalogLibrariesCallback = (libraries: GradleLibrary[]) =>
Expand Down
5 changes: 4 additions & 1 deletion generators/gradle/types.d.ts
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
import { RequireOneOrNone } from 'type-fest';

export type GradleScript = { script: string };

export type GradleDependency = { groupId: string; artifactId: string; version?: string; scope: string; classifier?: string };
Expand All @@ -10,7 +12,7 @@ export type GradleRepository = { url: string; username?: string; password?: stri

export type GradleTomlVersion = { name: string; version?: string };

export type GradleTomlAnyItemVersion = { version: string } | { ['version.ref']: string };
export type GradleTomlAnyItemVersion = RequireOneOrNone<{ version: string; 'version.ref': string }, 'version' | 'version.ref'>;

export type GradleTomlLibraryId = { module: string } | { group: string; name: string };

Expand All @@ -36,6 +38,7 @@ export type GradleSourceType = {
addGradleMavenRepository?(repository: GradleRepository): void;
addGradleBuildSrcDependency?(dependency: GradleDependency): void;
addGradleDependencyCatalogVersion?(catalogVersion: GradleTomlVersion): void;
addGradleDependencyCatalogVersions?(catalogVersion: GradleTomlVersion[]): void;
addGradleDependencyCatalogLibrary?(catalogVersion: GradleLibrary, options?: GradleNeedleOptions): void;
addGradleDependencyCatalogLibraries?(catalogVersion: GradleLibrary[], options?: GradleNeedleOptions): void;
addGradleDependencyCatalogPlugin?(catalogVersion: GradleTomlPlugin): void;
Expand Down
60 changes: 38 additions & 22 deletions generators/java/generator.ts
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ import {
generatedAnnotationTransform,
checkJava,
isReservedJavaKeyword,
mavenScopeToGradleScope,
javaScopeToGradleScope,
} from './support/index.js';
import command from './command.js';
import { JAVA_COMPATIBLE_VERSIONS } from '../generator-constants.js';
Expand All @@ -36,7 +36,8 @@ import { entityServerFiles, enumFiles } from './entity-files.js';
import { getEnumInfo } from '../base-application/support/index.js';
import { mutateData } from '../base/support/index.js';
import { javaBeanCase } from '../server/support/index.js';
import type { JavaArtifact } from './types.js';
import type { JavaDependency } from './types.js';
import type { MavenDependency } from '../maven/types.js';

export default class JavaGenerator extends BaseApplicationGenerator {
packageInfoFile!: boolean;
Expand Down Expand Up @@ -97,50 +98,65 @@ export default class JavaGenerator extends BaseApplicationGenerator {
const annotationProcessors = dependencies.filter(dep => dep.scope === 'annotationProcessor');
const importDependencies = dependencies.filter(dep => dep.scope === 'import');
const commonDependencies = dependencies.filter(dep => !['annotationProcessor', 'import'].includes(dep.scope!));
const convertVersionToProp = ({ version, artifactId, ...artifact }: JavaArtifact) => ({
...artifact,
artifactId,
version: version ? `\${${artifactId}.version}` : undefined,
});
const convertVersionToRef = ({ version, versionRef, ...artifact }: JavaDependency): MavenDependency =>
version || versionRef ? { ...artifact, version: `\${${versionRef ?? artifact.artifactId}.version}` } : artifact;
const removeScope = ({ scope: _scope, ...artifact }: MavenDependency) => artifact;

source.addMavenDefinition?.({
properties: dependencies
.filter(dep => dep.version)
.map(({ artifactId, version }) => ({ property: `${artifactId}.version`, value: version })),
dependencies: [
...commonDependencies.map(convertVersionToProp),
...commonDependencies.map(convertVersionToRef),
// Add a provided scope for annotation processors so that version is not required in annotationProcessor dependencies
...annotationProcessors
.filter(dep => !dep.version)
.map(({ scope: _scope, ...artifact }) => ({ ...artifact, scope: 'provided' })),
...annotationProcessors.filter(dep => !dep.version).map(artifact => ({ ...artifact, scope: 'provided' })),
],
dependencyManagement: importDependencies.map(convertVersionToProp),
annotationProcessors: annotationProcessors.map(convertVersionToProp).map(({ scope: _scope, ...artifact }) => artifact),
dependencyManagement: importDependencies.map(convertVersionToRef),
annotationProcessors: annotationProcessors.map(convertVersionToRef).map(removeScope),
});
}

if (application.buildToolGradle) {
source.addGradleDependencies?.(
dependencies
.filter(dep => !dep.version)
.filter(dep => !dep.version && !dep.versionRef)
.map(({ scope, type, ...artifact }) => ({
...artifact,
scope: mavenScopeToGradleScope({ scope, type }),
scope: javaScopeToGradleScope({ scope, type }),
})),
options,
);
source.addGradleDependencyCatalogLibraries?.(
dependencies
.filter(dep => dep.version)
.map(({ scope, type, groupId, artifactId, version }) => ({
libraryName: artifactId,
module: `${groupId}:${artifactId}`,
version: version!,
scope: mavenScopeToGradleScope({ scope, type }),
})),
.filter(dep => dep.version || dep.versionRef)
.map(({ scope, type, groupId, artifactId, version, versionRef }) => {
const library = {
libraryName: artifactId,
module: `${groupId}:${artifactId}`,
scope: javaScopeToGradleScope({ scope, type }),
};
return version ? { ...library, version } : { ...library, 'version.ref': versionRef! };
}),
);
}
};

source.addJavaDefinition = (definition, options) => {
const { dependencies, versions } = definition;
if (dependencies) {
source.addJavaDependencies!(dependencies, options);
}
if (versions) {
if (application.buildToolMaven) {
source.addMavenDefinition!({
properties: versions.map(({ name, version }) => ({ property: `${name}.version`, value: version })),
});
}
if (application.buildToolGradle) {
source.addGradleDependencyCatalogVersions?.(versions);
}
}
};
},
});
}
Expand Down
2 changes: 1 addition & 1 deletion generators/java/support/artifacts.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { JavaArtifactType } from '../types.js';

export const mavenScopeToGradleScope = (artifactType: JavaArtifactType): string => {
export const javaScopeToGradleScope = (artifactType: JavaArtifactType): string => {
const { scope = 'compile', type = 'jar' } = artifactType;
if (type === 'pom') {
if (scope === 'import') {
Expand Down
19 changes: 17 additions & 2 deletions generators/java/types.d.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,12 @@
import { RequireOneOrNone } from 'type-fest';
import { BaseApplication } from '../base-application/types.js';
import { GradleNeedleOptions } from '../gradle/types.js';

export type JavaDependencyVersion = {
name: string;
version: string;
};

export type JavaArtifactType = {
type?: 'jar' | 'pom';
scope?: 'compile' | 'provided' | 'runtime' | 'test' | 'system' | 'import' | 'annotationProcessor';
Expand All @@ -9,10 +15,18 @@ export type JavaArtifactType = {
export type JavaArtifact = {
groupId: string;
artifactId: string;
version?: string;
classifier?: string;
} & JavaArtifactType;

export type JavaArtifactVersion = RequireOneOrNone<{ version: string; versionRef: string }, 'version' | 'versionRef'>;

export type JavaDependency = JavaArtifact & JavaArtifactVersion;

export type JavaDefinition = {
versions: JavaDependencyVersion[];
dependencies: JavaDependency[];
};

export type JavaNeedleOptions = GradleNeedleOptions;

export type JavaApplication = BaseApplication & {
Expand Down Expand Up @@ -42,5 +56,6 @@ export type JavaApplication = BaseApplication & {
};

export type JavaSourceType = {
addJavaDependencies?(dependency: JavaArtifact[], options?: JavaNeedleOptions): void;
addJavaDefinition?(definition: JavaDefinition, options?: JavaNeedleOptions): void;
addJavaDependencies?(dependency: JavaDependency[], options?: JavaNeedleOptions): void;
};
2 changes: 1 addition & 1 deletion generators/spring-boot/generator.ts
Original file line number Diff line number Diff line change
Expand Up @@ -405,7 +405,7 @@ public void set${javaBeanCase(propertyName)}(${propertyType} ${propertyName}) {
{
groupId: 'tech.jhipster',
artifactId: 'jhipster-dependencies',
version: application.jhipsterDependenciesVersion,
version: application.jhipsterDependenciesVersion!,
type: 'pom',
scope: 'import',
},
Expand Down
4 changes: 2 additions & 2 deletions generators/spring-cache/generator.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ import writeTask from './files.js';
import cleanupTask from './cleanup.js';
import { createNeedleCallback } from '../base/support/needles.js';
import { getCacheProviderMavenDefinition } from './internal/dependencies.js';
import { JavaArtifact } from '../java/types.js';
import { JavaDependency } from '../java/types.js';

export default class SpringCacheGenerator extends BaseApplicationGenerator {
async beforeQueue() {
Expand Down Expand Up @@ -135,7 +135,7 @@ export default class SpringCacheGenerator extends BaseApplicationGenerator {
const { javaDependencies } = application;
const { cacheProvider, enableHibernateCache } = application as any;

const dependencies: JavaArtifact[] = [{ groupId: 'org.springframework.boot', artifactId: 'spring-boot-starter-cache' }];
const dependencies: JavaDependency[] = [{ groupId: 'org.springframework.boot', artifactId: 'spring-boot-starter-cache' }];
const definition = getCacheProviderMavenDefinition(cacheProvider, javaDependencies);
dependencies.push(...definition.base.dependencies);
if (enableHibernateCache && definition.hibernateCache) {
Expand Down
6 changes: 3 additions & 3 deletions generators/spring-cache/internal/dependencies.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
* limitations under the License.
*/

import { JavaArtifact } from '../../java/types.js';
import { JavaDependency } from '../../java/types.js';

const javaxCacheApi = {
groupId: 'javax.cache',
Expand All @@ -29,8 +29,8 @@ const hibernateJCache = {
};

type CacheProviderDependencies = {
base: { dependencies: JavaArtifact[] };
hibernateCache?: { dependencies: JavaArtifact[] };
base: { dependencies: JavaDependency[] };
hibernateCache?: { dependencies: JavaDependency[] };
};

// eslint-disable-next-line import/prefer-default-export
Expand Down
4 changes: 2 additions & 2 deletions generators/spring-data-relational/internal/dependencies.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ const testcontainerFileForDB = {
postgresql: 'PostgreSqlTestContainer.java',
};

type JavaArtifact = { groupId: string; artifactId: string; version?: string };
export type DatabaseArtifact = { jdbc: JavaArtifact; r2dbc: JavaArtifact };
type JavaDependency = { groupId: string; artifactId: string; version?: string };
export type DatabaseArtifact = { jdbc: JavaDependency; r2dbc: JavaDependency };

const databaseArtifactForDB: Record<string, DatabaseArtifact> = {
mariadb: {
Expand Down

0 comments on commit 790909c

Please sign in to comment.