declapract-typescript-ehmpathy 0.47.71 → 0.47.72

package/dist/practices/environments/bad-practices/old-stage-enum/src/<star><star>/<star>.ts.declapract.ts

@@ -17,7 +17,7 @@ export const check: FileCheckFunction = (contents) => {
 export const fix: FileFixFunction = (contents) => {
   if (!contents) return { contents };
 
-  let fixed = contents
+  const fixed = contents
     // replace enum values with string literals
     .replace(/Stage\.PRODUCTION/g, "'prod'")
     .replace(/Stage\.DEVELOPMENT/g, "'prep'")

package/dist/practices/provision-github/bad-practices/deprecated-reviewers-variable/declapract.use.yml.declapract.ts

@@ -0,0 +1,41 @@
+import type { FileCheckFunction, FileFixFunction } from 'declapract';
+import yaml from 'yaml';
+
+/**
+ * .what = removes deprecated reviewers.users variable from declapract.use.yml
+ * .why = production-on-else environment now uses team:releaser instead of user list
+ */
+
+export const check: FileCheckFunction = (contents) => {
+  if (!contents) throw new Error('no file found');
+
+  const parsed = yaml.parse(contents) as {
+    variables?: Record<string, unknown>;
+  };
+  const variables = parsed?.variables ?? {};
+
+  // check if reviewers or reviewers.users exists
+  if ('reviewers' in variables) return; // bad practice detected
+
+  throw new Error('no deprecated reviewers variable found');
+};
+
+export const fix: FileFixFunction = (contents) => {
+  if (!contents) return { contents };
+
+  // parse with parseDocument to preserve comments and format
+  const doc = yaml.parseDocument(contents);
+  const parsed = doc.toJSON() as { variables?: Record<string, unknown> };
+
+  if (!parsed?.variables || !('reviewers' in parsed.variables)) {
+    return { contents };
+  }
+
+  // remove the reviewers key from variables
+  const variablesNode = doc.get('variables') as yaml.YAMLMap | undefined;
+  if (variablesNode) {
+    variablesNode.delete('reviewers');
+  }
+
+  return { contents: doc.toString() };
+};

package/dist/practices/provision-github/best-practice/provision/github.repo/resources.ts

@@ -134,7 +134,7 @@ export const getResources = async (): Promise<DomainEntity<any>[]> => {
   const envProductionOnElse = DeclaredGithubEnvironment.as({
     repo,
     name: 'production-on-else',
-    reviewers: { users: @declapract{variable.reviewers.users}, teams: null },
+    reviewers: { users: null, teams: ['releasers'] },
     waitTimer: null, // no delay once approved
     deploymentBranchPolicy: null, // any branch
     preventSelfReview: false, // self-approval allowed if in reviewers list

package/package.json

@@ -2,7 +2,7 @@
   "name": "declapract-typescript-ehmpathy",
   "author": "ehmpathy",
   "description": "declapract best practices declarations for typescript",
-  "version": "0.47.71",
+  "version": "0.47.72",
   "license": "MIT",
   "main": "src/index.js",
   "repository": "ehmpathy/declapract-typescript-ehmpathy",