fix: match variable aliases using transformed constant names

src/commands/diff/diff.test.ts

@@ -292,6 +292,27 @@ describe('diff', () => {
         .it('identifies aliased variables specified in config file', (ctx) => {
             expect(ctx.stdout).toMatchSnapshot()
         })
+
+    test.stdout()
+        .command([
+            'diff',
+            '--file',
+            'test-utils/fixtures/diff/auto-alias-transform',
+            '--no-api',
+            '--repo-config-path',
+            './test-utils/fixtures/configs/autoAliasConfig.yml',
+        ])
+        .it(
+            'resolves aliases from generated types file using key transformation',
+            (ctx) => {
+                // Should NOT show "could not be identified" warnings
+                expect(ctx.stdout).not.toContain('could not be identified')
+                // Should identify the variables
+                expect(ctx.stdout).toContain('my-test-variable')
+                expect(ctx.stdout).toContain('enable-feature')
+                expect(ctx.stdout).toContain('removed-variable')
+            },
+        )
     test.stdout()
         .command([
             'diff',

src/commands/diff/index.ts

@@ -14,6 +14,7 @@ import VarAliasFlag, { getVariableAliases } from '../../flags/var-alias'
 import ShowRegexFlag, { showRegex } from '../../flags/show-regex'
 import { Variable } from '../../api/schemas'
 import { FileFilters } from '../../utils/FileFilters'
+import { keyToConstant } from '../../utils/keyToConstant'
 
 const EMOJI = {
     add: '🟢',
@@ -162,7 +163,11 @@ export default class Diff extends Base {
         Object.values(matchesBySdk).forEach((matches) => {
             matches.forEach((m) => {
                 const match = { ...m }
-                const aliasedName = aliasMap[match.name]
+                // Try to find alias by direct match first, then by transformed constant name
+                // This handles cases where the generated types file creates constants like MY_VARIABLE
+                // for a variable key like 'my-variable'
+                const aliasedName =
+                    aliasMap[match.name] || aliasMap[keyToConstant(match.name)]
                 if (match.isUnknown && aliasedName) {
                     match.alias = match.name
                     match.name = aliasedName

src/commands/generate/types.ts

@@ -4,9 +4,9 @@ import { fetchAllVariables } from '../../api/variables'
 import { Flags } from '@oclif/core'
 import { Feature, Project, Variable, CustomProperty } from '../../api/schemas'
 import { OrganizationMember, fetchOrganizationMembers } from '../../api/members'
-import { upperCase } from 'lodash'
 import { createHash } from 'crypto'
 import path from 'path'
+import { keyToConstant } from '../../utils/keyToConstant'
 import {
     fetchAllCompletedOrArchivedFeatures,
     fetchFeatures,
@@ -353,7 +353,7 @@ export const ${constantName} = '${hashedKey}' as const`
     }
 
     getVariableGeneratedName(variable: Variable) {
-        let constantName = upperCase(variable.key).replace(/\s/g, '_')
+        let constantName = keyToConstant(variable.key)
 
         if (this.methodNames[constantName]?.length) {
             constantName = `${constantName}_${this.methodNames[constantName].length}`

src/utils/keyToConstant.ts

@@ -0,0 +1,10 @@
+import { upperCase } from 'lodash'
+
+/**
+ * Converts a variable key to its generated constant name format.
+ *
+ * Example: 'my-variable' -> 'MY_VARIABLE'
+ */
+export function keyToConstant(variableKey: string): string {
+    return upperCase(variableKey).replace(/\s/g, '_')
+}

test-utils/fixtures/configs/autoAliasConfig.yml

@@ -0,0 +1,13 @@
+project: "test-project"
+org:
+  id: "org_test"
+  name: "test-org"
+
+typeGenerator:
+  outputPath: "./test-utils/fixtures/configs/mockTypes.ts"
+
+codeInsights:
+  matchPatterns:
+    ts:
+      - ":\\s*([a-z][a-z0-9-]*)"
+

test-utils/fixtures/configs/mockTypes.ts

@@ -0,0 +1,5 @@
+// Mock generated types file for testing auto-alias resolution
+export const MY_TEST_VARIABLE = 'my-test-variable' as const
+export const ENABLE_FEATURE = 'enable-feature' as const
+export const REMOVED_VARIABLE = 'removed-variable' as const
+

test-utils/fixtures/diff/auto-alias-transform

@@ -0,0 +1,17 @@
+diff --git a/test-utils/fixtures/diff/sampleDiff b/test-utils/fixtures/diff/sampleDiff
+new file mode 100644
+index 00000000..e69de29b
+diff --git a/test-utils/fixtures/diff/sampleDiff.ts b/test-utils/fixtures/diff/sampleDiff.ts
+new file mode 100644
+index 00000000..ed8ee4ab
+--- /dev/null
++++ b/test-utils/fixtures/diff/sampleDiff.ts
+@@ -1,1 +1,21 @@
++const config = {
++  feature: my-test-variable,
++  beta: enable-feature
++}
+-const old = {
+-  feature: removed-variable
+-}
+