@rivinia/env 0.3.1 → 0.3.2

package/README.md

@@ -8,7 +8,7 @@ Angular CLI `ng add` schematic. It does not read environment values at
 runtime: Angular's build file replacements select a statically imported
 environment file.
 
-- Prepared patch: **0.3.1** (latest published: **0.3.0**)
+- Package version: **0.3.2**
 - No runtime providers or services; the package supplies a type and CLI tooling.
 - The generated model imports `BaseEnvironment` from this package.
 
@@ -72,12 +72,16 @@ their generated API URLs are `https://staging.example.com/api` and
 `https://api.example.com/api`. Replace these example URLs and any other
 generated values with the app's real configuration.
 
-The schematic adds a file replacement to an existing `production`
-configuration when `prod` is selected. When `stage` is selected, it creates a
-`staging` configuration if needed (copying `production` settings except its
-`fileReplacements` and `outputPath`) and adds the staging replacement.
-`development` intentionally uses `environment.ts` directly. The schematic
-does not create a missing `production` or `development` configuration.
+The schematic creates the short `dev`, `stag`, and `prod` build configurations
+for the selected environments, copying settings from the corresponding
+`development`, `staging`, or `production` configuration when one exists.
+Otherwise, it creates the short configuration with the required environment
+replacement when needed (`dev` uses the base file directly). It also wires
+existing legacy keys when present: `development`
+uses `environment.ts`, `staging` replaces it with
+`environment.staging.ts`, and `production` replaces it with
+`environment.prod.ts`. The short names are the recommended CLI commands; the
+legacy names remain available for existing scripts.
 
 For selected configurations that exist, it sets `outputPath` to
 `builds/dev`, `builds/stag`, or `builds/prod` only if no output path is
@@ -97,24 +101,28 @@ Always import the base path from the app, not a stage/prod-specific file:
 
 ```ts
 // src/app/trip.service.ts
-import { Injectable } from '@angular/core';
+import { Service } from '@angular/core';
 import { environment } from '../environments/environment';
 
-@Injectable({ providedIn: 'root' })
+@Service()
 export class TripService {
   readonly apiUrl = environment.apiUrl;
 }
 ```
 
-The Angular CLI replaces `environment.ts` for the selected build
-configuration. For example:
+After `ng add @rivinia/env`, build with the short configuration names:
 
 ```bash
-ng build --configuration development
-ng build --configuration staging
-ng build --configuration production
+ng build --configuration dev
+ng build --configuration stag
+ng build --configuration prod
 ```
 
+These are actual keys in `angular.json`, not Angular CLI aliases. If you
+maintain custom configurations, inspect the selected project's
+`architect.build.configurations` and ensure each key has the intended
+`fileReplacements`; `ng build --configuration` uses the key exactly as written.
+
 The generated files are compile-time configuration, not secrets: values
 included in a browser build are visible to its users. `BaseEnvironment`
 requires only readonly `production: boolean` and `version: string`; extend

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rivinia/env",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "Typed application environments for Angular (dev/staging/prod) with an ng-add schematic.",
   "keywords": [
     "angular",

package/schematics/ng-add/index.cjs

@@ -28,8 +28,14 @@ const OUTPUT_DIR = {
     stage: 'builds/stag',
     prod: 'builds/prod',
 };
-/** Maps an environment name to its Angular CLI configuration name. */
+/** Public short configuration names used by `ng build --configuration`. */
 const CONFIG_NAME = {
+    dev: 'dev',
+    stage: 'stag',
+    prod: 'prod',
+};
+/** Existing long configuration names remain wired for compatibility. */
+const LEGACY_CONFIG_NAME = {
     dev: 'development',
     stage: 'staging',
     prod: 'production',
@@ -38,11 +44,11 @@ const fileDefs = (sourceRoot, selected) => {
     const files = [
         { name: `${sourceRoot}/environments/environment.model.ts`, content: () => MODEL_FILE },
         // The base file is always scaffolded: it is the `replace` source for
-        // prod/staging swaps (and what `development` uses directly).
+        // prod/stag swaps (and what dev uses directly).
         {
             name: `${sourceRoot}/environments/environment.ts`,
             content: (version) => `import type { AppEnvironment } from './environment.model';\n\n` +
-                `/** Development defaults. Used as-is by the \`development\` configuration. */\n` +
+                `/** Development defaults. Used as-is by the \`dev\` configuration. */\n` +
                 `export const environment: AppEnvironment = {\n` +
                 `  production: false,\n` +
                 `  version: '${version}',\n` +
@@ -54,7 +60,7 @@ const fileDefs = (sourceRoot, selected) => {
         files.push({
             name: `${sourceRoot}/environments/environment.staging.ts`,
             content: (version) => `import type { AppEnvironment } from './environment.model';\n\n` +
-                `/** Staging mirrors production. Wired via the \`staging\` configuration. */\n` +
+                `/** Staging mirrors production. Wired via the \`stag\` configuration. */\n` +
                 `export const environment: AppEnvironment = {\n` +
                 `  production: true,\n` +
                 `  version: '${version}',\n` +
@@ -77,14 +83,15 @@ const fileDefs = (sourceRoot, selected) => {
 };
 /**
  * `ng add @rivinia/env` — scaffolds typed environment files and wires
- * development/staging/production configurations with file replacements.
+ * dev/stag/prod configurations with file replacements.
  *
  * - Never overwrites existing files (each file skipped individually).
  * - Never removes user configuration: only ADDS missing fileReplacements
- *   (matched by replace+with pair) and creates a `staging` configuration
- *   (copied from `production` when present) if absent.
- * - `development` intentionally gets no replacement (CLI standard: it uses
- *   `environment.ts` directly).
+ *   (one per source file) and creates short configurations from their legacy
+ *   equivalents when absent.
+ * - `dev` intentionally gets no replacement (it uses `environment.ts`).
+ *   Existing `development`, `staging`, and `production` configurations remain
+ *   wired for compatibility.
  * - App version is snapshotted from the app's `package.json` (`0.0.0` fallback).
  */
 function ngAdd(options) {
@@ -127,14 +134,20 @@ function ngAdd(options) {
             created++;
         }
         if (selected.includes('prod')) {
-            ensureReplacement(tree, angular, name, 'production', {
+            ensureNamedConfig(tree, angular, name, 'prod', ['production'], context);
+            const replacement = {
                 replace: `${sourceRoot}/environments/environment.ts`,
                 with: `${sourceRoot}/environments/environment.prod.ts`,
-            });
+            };
+            ensureReplacement(tree, angular, name, 'prod', replacement);
+            ensureReplacement(tree, angular, name, 'production', replacement);
         }
         if (selected.includes('stage')) {
             ensureStaging(tree, angular, name, sourceRoot, context);
         }
+        if (selected.includes('dev')) {
+            ensureNamedConfig(tree, angular, name, 'dev', ['development'], context);
+        }
         ensureOutputPaths(tree, angular, name, selected, context);
         if (created > 0 || configsSnapshot() !== configsBefore) {
             tree.overwrite('/angular.json', JSON.stringify(angular, null, 2) + '\n');
@@ -204,18 +217,25 @@ function ensureReplacement(tree, angular, name, configuration, entry) {
     const configurations = getConfigurations(angular, name);
     const target = configurations[configuration];
     if (!target)
-        return; // never create production/development for the user
+        return;
     const list = target['fileReplacements'];
     if (!Array.isArray(list)) {
         target['fileReplacements'] = [{ ...entry }];
         return;
     }
-    const exists = list.some((item) => typeof item === 'object' &&
+    const matches = list
+        .map((item, index) => ({ item, index }))
+        .filter(({ item }) => typeof item === 'object' &&
         item !== null &&
-        item['replace'] === entry.replace &&
-        item['with'] === entry.with);
-    if (!exists)
+        item['replace'] === entry.replace);
+    if (matches.length === 0) {
         list.push({ ...entry });
+        return;
+    }
+    const firstIndex = matches[0].index;
+    list[firstIndex] = { ...entry };
+    for (const { index } of matches.slice(1).reverse())
+        list.splice(index, 1);
 }
 /**
  * Points each selected configuration at its build directory
@@ -228,12 +248,13 @@ function ensureOutputPaths(tree, angular, name, selected, context) {
     const configurations = getConfigurations(angular, name);
     let touched = false;
     for (const env of selected) {
-        const configName = CONFIG_NAME[env];
-        const target = configurations[configName];
-        if (!target || 'outputPath' in target)
-            continue;
-        target['outputPath'] = OUTPUT_DIR[env];
-        touched = true;
+        for (const configName of [CONFIG_NAME[env], LEGACY_CONFIG_NAME[env]]) {
+            const target = configurations[configName];
+            if (!target || 'outputPath' in target)
+                continue;
+            target['outputPath'] = OUTPUT_DIR[env];
+            touched = true;
+        }
     }
     if (touched)
         ensureGitignore(tree, context);
@@ -251,17 +272,28 @@ function ensureGitignore(tree, context) {
 function ensureStaging(tree, angular, name, sourceRoot, context) {
     if (!tree.exists('/angular.json'))
         return;
-    const configurations = getConfigurations(angular, name);
-    if (!configurations['staging']) {
-        const production = configurations['production'];
-        const copy = production && typeof production === 'object' ? { ...production } : {};
-        delete copy['fileReplacements'];
-        delete copy['outputPath'];
-        configurations['staging'] = copy;
-        context.logger.info('[rivinia/env] created `staging` configuration.');
-    }
-    ensureReplacement(tree, angular, name, 'staging', {
+    ensureNamedConfig(tree, angular, name, 'stag', ['staging', 'production'], context);
+    ensureNamedConfig(tree, angular, name, 'staging', ['stag', 'production'], context);
+    const replacement = {
         replace: `${sourceRoot}/environments/environment.ts`,
         with: `${sourceRoot}/environments/environment.staging.ts`,
-    });
+    };
+    ensureReplacement(tree, angular, name, 'stag', replacement);
+    ensureReplacement(tree, angular, name, 'staging', replacement);
+}
+function ensureNamedConfig(tree, angular, name, configuration, fallbacks, context) {
+    if (!tree.exists('/angular.json'))
+        return;
+    const configurations = getConfigurations(angular, name);
+    if (configurations[configuration])
+        return;
+    const source = fallbacks
+        .map((fallback) => configurations[fallback])
+        .find((candidate) => candidate && typeof candidate === 'object');
+    const copy = source
+        ? JSON.parse(JSON.stringify(source))
+        : {};
+    delete copy['outputPath'];
+    configurations[configuration] = copy;
+    context.logger.info(`[rivinia/env] created \`${configuration}\` configuration.`);
 }