Refine commands and clean up unused ones (#61)

* Clean up unused commands

* move rimraf to dev dependency

* update usage accordingly

* update test in pipeline

Author: jojozhuang

Dockerfile

@@ -18,7 +18,7 @@ WORKDIR /ng-app
 COPY package*.json tsconfig*.json angular.json ./
 COPY ./src ./src
 
-RUN npm ci --quiet && npm run build-nas
+RUN npm ci --quiet && npm run build:nas
 
 # Stage 2, based on Nginx, to have only the compiled app, ready for production with Nginx
 

Dockerfile-nas

@@ -15,7 +15,7 @@ WORKDIR /ng-app
 COPY package*.json tsconfig*.json angular.json ./
 COPY ./src ./src
 
-RUN npm ci --quiet && npm run build-nas
+RUN npm ci --quiet && npm run build:nas
 
 # Stage 2, based on Nginx, to have only the compiled app, ready for production with Nginx
 

Dockerfile-prod

@@ -15,7 +15,7 @@ WORKDIR /ng-app
 COPY package*.json tsconfig*.json angular.json ./
 COPY ./src ./src
 
-RUN npm ci --quiet && npm run build:aot:prod
+RUN npm ci --quiet && npm run build:cross:env
 
 # Stage 2, based on Nginx, to have only the compiled app, ready for production with Nginx
 

Dockerfile2

@@ -22,7 +22,7 @@ WORKDIR /ng-app
 
 COPY . .
 
-RUN npm run build:aot:prod
+RUN npm run build:cross:env
 
 # Stage 2, based on Nginx, to have only the compiled app, ready for production with Nginx
 

azure-pipelines.yml

@@ -101,7 +101,7 @@ stages:
 
           - script: |
               # Run unit tests in headless Chrome and produce coverage
-              npm run test-headless -- --no-watch --browsers=ChromeHeadless
+              npm run test:headless
             displayName: 'Run unit tests (headless)'
 
   - stage: Build

package-lock.json

@@ -7,15 +7,15 @@
   },
   "license": "MIT",
   "scripts": {
-    "start": "node server.js",
-    "dev": "ng serve",
-    "build-aot": "ng build --aot --prod",
+    "start": "ng serve",
+    "start:prod": "ng serve --configuration production",
+    "start:nas": "ng serve --configuration nas",
+    "start:mock": "node server.js",
     "build": "ng build --configuration production",
-    "build:aot:prod": "rimraf dist compiled && cross-env BUILD_AOT=1 SOURCE_MAP=0 ng build --prod",
-    "build-nas": "ng build --configuration nas",
-    "start-nas": "ng serve --configuration nas",
+    "build:nas": "ng build --configuration nas",
+    "build:cross:env": "rimraf dist compiled && cross-env SOURCE_MAP=0 ng build --configuration production",
     "test": "ng test",
-    "test-headless": "ng test --watch=false --browsers=ChromeHeadless",
+    "test:headless": "ng test --watch=false --browsers=ChromeHeadless",
     "lint": "eslint '*/**/*.{js,ts,tsx}' --quiet --fix",
     "e2e": "ng e2e"
   },
@@ -33,7 +33,6 @@
     "@angular/router": "^20.3.6",
     "bootstrap": "^4.6.1",
     "ngx-monaco-editor-v2": "^20.3.0",
-    "rimraf": "^2.6.3",
     "rxjs": "^6.6.7",
     "rxjs-compat": "^6.6.7",
     "tslib": "^2.3.1",
@@ -62,6 +61,7 @@
     "karma-jasmine-html-reporter": "~2.1.0",
     "prettier": "^3.6.2",
     "protractor": "~7.0.0",
+    "rimraf": "^2.6.3",
     "ts-node": "~4.1.0",
     "typescript": "~5.8.3",
     "typescript-eslint": "8.46.0"

package.json

@@ -25,8 +25,8 @@ describe('TextCompareComponent', () => {
   it('should render title in a h1 tag', waitForAsync(async () => {
     const fixture = TestBed.createComponent(CompareEditorComponent);
     fixture.detectChanges();
-    const compiled = fixture.debugElement.nativeElement as HTMLElement;
-    const h1Element = compiled.querySelector('h1');
+    const compareEditor = fixture.debugElement.nativeElement as HTMLElement;
+    const h1Element = compareEditor.querySelector('h1');
     await expect(h1Element).toBeTruthy('h1 element should exist');
     if (h1Element) {
       await expect(h1Element.textContent).toContain('Text Compare');