refactor(@angular-devkit/architect): update to support building with isolated declarations

The `@angular-devkit/architect` package can now be built with the TypeScript
`isolatedDeclarations` option enabled. The changes were type only and
mainly related to function return types and module level variable types
additions.

Author: clydin

packages/angular_devkit/architect/builders/all-of.ts

@@ -6,12 +6,11 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import { json } from '@angular-devkit/core';
 import { EMPTY, from, map, mergeMap, of } from 'rxjs';
-import { BuilderOutput, BuilderRun, createBuilder } from '../src';
+import { Builder, BuilderOutput, BuilderRun, createBuilder } from '../src';
 import { Schema as OperatorSchema } from './operator-schema';
 
-export default createBuilder<json.JsonObject & OperatorSchema>((options, context) => {
+const builder: Builder<OperatorSchema> = createBuilder((options, context) => {
   const allRuns: Promise<[number, BuilderRun]>[] = [];
 
   context.reportProgress(
@@ -66,3 +65,5 @@ export default createBuilder<json.JsonObject & OperatorSchema>((options, context
     }),
   );
 });
+
+export default builder;

packages/angular_devkit/architect/builders/concat.ts

@@ -6,12 +6,11 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import { json } from '@angular-devkit/core';
 import { concatMap, first, from, last, map, of, switchMap } from 'rxjs';
-import { BuilderOutput, BuilderRun, createBuilder } from '../src';
+import { Builder, BuilderOutput, BuilderRun, createBuilder } from '../src';
 import { Schema as OperatorSchema } from './operator-schema';
 
-export default createBuilder<json.JsonObject & OperatorSchema>((options, context) => {
+const builder: Builder<OperatorSchema> = createBuilder((options, context) => {
   const allRuns: (() => Promise<BuilderRun>)[] = [];
 
   context.reportProgress(
@@ -61,3 +60,5 @@ export default createBuilder<json.JsonObject & OperatorSchema>((options, context
     last(),
   );
 });
+
+export default builder;

packages/angular_devkit/architect/builders/false.ts

@@ -6,9 +6,11 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import { createBuilder } from '../src';
+import { type Builder, createBuilder } from '../src';
 
-export default createBuilder(() => ({
+const builder: Builder<{}> = createBuilder(() => ({
   success: false,
   error: 'False builder always errors.',
 }));
+
+export default builder;

packages/angular_devkit/architect/builders/true.ts

@@ -6,6 +6,8 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import { createBuilder } from '../src';
+import { type Builder, createBuilder } from '../src';
 
-export default createBuilder(() => ({ success: true }));
+const builder: Builder<{}> = createBuilder(() => ({ success: true }));
+
+export default builder;

packages/angular_devkit/architect/node/node-modules-architect-host.ts

@@ -121,7 +121,7 @@ export class WorkspaceNodeModulesArchitectHost implements ArchitectHost<NodeModu
     }
   }
 
-  async getBuilderNameForTarget(target: Target) {
+  async getBuilderNameForTarget(target: Target): Promise<string> {
     return this.workspaceHost.getBuilderName(target.project, target.target);
   }
 
@@ -134,7 +134,7 @@ export class WorkspaceNodeModulesArchitectHost implements ArchitectHost<NodeModu
    */
   resolveBuilder(
     builderStr: string,
-    basePath = this._root,
+    basePath: string = this._root,
     seenBuilders?: Set<string>,
   ): Promise<NodeModulesBuilderInfo> {
     if (seenBuilders?.has(builderStr)) {
@@ -228,11 +228,11 @@ export class WorkspaceNodeModulesArchitectHost implements ArchitectHost<NodeModu
     });
   }
 
-  async getCurrentDirectory() {
+  async getCurrentDirectory(): Promise<string> {
     return process.cwd();
   }
 
-  async getWorkspaceRoot() {
+  async getWorkspaceRoot(): Promise<string> {
     return this._root;
   }
 

packages/angular_devkit/architect/src/architect.ts

@@ -364,7 +364,7 @@ export class Architect {
     this._scheduler = new SimpleScheduler(jobRegistry, registry);
   }
 
-  has(name: JobName) {
+  has(name: JobName): Observable<boolean> {
     return this._scheduler.has(name);
   }
 

packages/angular_devkit/architect/src/internal.ts

@@ -19,15 +19,15 @@ import { JobDescription, JobHandler } from './jobs';
  * Using Symbol.for() as it's a global registry that's the same for all installations of
  * Architect (if some libraries depends directly on architect instead of sharing the files).
  */
-export const BuilderSymbol = Symbol.for('@angular-devkit/architect:builder');
+export const BuilderSymbol: unique symbol = Symbol.for('@angular-devkit/architect:builder');
 
 /**
  * BuilderVersionSymbol used for knowing which version of the library createBuilder() came from.
  * This is to make sure we don't try to use an incompatible builder.
  * Using Symbol.for() as it's a global registry that's the same for all installations of
  * Architect (if some libraries depends directly on architect instead of sharing the files).
  */
-export const BuilderVersionSymbol = Symbol.for('@angular-devkit/architect:version');
+export const BuilderVersionSymbol: unique symbol = Symbol.for('@angular-devkit/architect:version');
 
 /**
  * A Specialization of the JobHandler type. This exposes BuilderDescription as the job description

packages/angular_devkit/architect/src/jobs/fallback-registry.ts

@@ -27,7 +27,7 @@ export class FallbackRegistry<
     >[] = [],
   ) {}
 
-  addFallback(registry: Registry) {
+  addFallback(registry: Registry): void {
     this._fallbacks.push(registry);
   }
 

packages/angular_devkit/architect/src/jobs/simple-scheduler.ts

@@ -185,7 +185,7 @@ export class SimpleScheduler<
    * @param name The name of the job.
    * @returns A description, or null if the job is not registered.
    */
-  getDescription(name: JobName) {
+  getDescription(name: JobName): Observable<JobDescription | null> {
     return concat(
       this._getInternalDescription(name).pipe(map((x) => x && x.jobDescription)),
       of(null),
@@ -197,7 +197,7 @@ export class SimpleScheduler<
    * @param name The name of the job.
    * @returns True if the job exists, false otherwise.
    */
-  has(name: JobName) {
+  has(name: JobName): Observable<boolean> {
     return this.getDescription(name).pipe(map((x) => x !== null));
   }
 
@@ -209,7 +209,7 @@ export class SimpleScheduler<
    *
    * Jobs already running are NOT paused. This is pausing the scheduler only.
    */
-  pause() {
+  pause(): () => void {
     let called = false;
     this._pauseCounter++;
 

packages/angular_devkit/architect/testing/test-project-host.ts

@@ -126,15 +126,15 @@ export class TestProjectHost extends NodeJsSyncHost {
     });
   }
 
-  replaceInFile(path: string, match: RegExp | string, replacement: string) {
+  replaceInFile(path: string, match: RegExp | string, replacement: string): void {
     const content = virtualFs.fileBufferToString(this.scopedSync().read(normalize(path)));
     this.scopedSync().write(
       normalize(path),
       virtualFs.stringToFileBuffer(content.replace(match, replacement)),
     );
   }
 
-  appendToFile(path: string, str: string) {
+  appendToFile(path: string, str: string): void {
     const content = virtualFs.fileBufferToString(this.scopedSync().read(normalize(path)));
     this.scopedSync().write(normalize(path), virtualFs.stringToFileBuffer(content.concat(str)));
   }
@@ -147,7 +147,7 @@ export class TestProjectHost extends NodeJsSyncHost {
     return fileName || undefined;
   }
 
-  copyFile(from: string, to: string) {
+  copyFile(from: string, to: string): void {
     const content = this.scopedSync().read(normalize(from));
     this.scopedSync().write(normalize(to), content);
   }