@codeyam/codeyam-cli 0.1.0-bleeding-edge.8afd3ee → 0.1.0-staging.1

package/codeyam-cli/src/webserver/build/server/index.js

@@ -1 +1 @@
-import{W as I,N as J,O as K,U as L,Q as X,R as Y,M as Z,T as _,V as $}from"./assets/server-build-Bi1mj14J.js";import"react/jsx-runtime";import"node:stream";import"@remix-run/node";import"@remix-run/react";import"isbot";import"react-dom/server";import"react";import"lucide-react";import"fetch-retry";import"better-sqlite3";import"pg";import"fs";import"path";import"kysely";import"kysely/helpers/sqlite";import"kysely/helpers/postgres";import"typescript";import"fs/promises";import"os";import"prompts";import"chalk";import"crypto";import"react-resizable";import"openai";import"p-queue";import"p-retry";import"child_process";import"util";import"@aws-sdk/client-dynamodb";import"json5";import"lru-cache";import"pluralize";import"piscina";import"@aws-sdk/util-dynamodb";import"dotenv";import"events";import"uuid";import"url";import"react-syntax-highlighter";import"react-syntax-highlighter/dist/cjs/styles/prism/index.js";import"node:crypto";import"v8";import"react-diff-viewer-continued";export{I as assets,J as assetsBuildDirectory,K as basename,L as entry,X as future,Y as isSpaMode,Z as mode,_ as publicPath,$ as routes};
+import{W as I,N as J,O as K,U as L,Q as X,R as Y,M as Z,T as _,V as $}from"./assets/server-build-lutv16q5.js";import"react/jsx-runtime";import"node:stream";import"@remix-run/node";import"@remix-run/react";import"isbot";import"react-dom/server";import"react";import"lucide-react";import"fetch-retry";import"better-sqlite3";import"pg";import"fs";import"path";import"kysely";import"kysely/helpers/sqlite";import"kysely/helpers/postgres";import"typescript";import"fs/promises";import"os";import"prompts";import"chalk";import"crypto";import"react-resizable";import"openai";import"p-queue";import"p-retry";import"child_process";import"util";import"@aws-sdk/client-dynamodb";import"json5";import"lru-cache";import"pluralize";import"piscina";import"@aws-sdk/util-dynamodb";import"dotenv";import"events";import"uuid";import"url";import"react-syntax-highlighter";import"react-syntax-highlighter/dist/cjs/styles/prism/index.js";import"node:crypto";import"v8";import"react-diff-viewer-continued";export{I as assets,J as assetsBuildDirectory,K as basename,L as entry,X as future,Y as isSpaMode,Z as mode,_ as publicPath,$ as routes};

package/codeyam-cli/src/webserver/build-info.json

@@ -1,7 +1,7 @@
 {
-  "buildTimestamp": "2025-12-09T21:06:53.801Z",
-  "buildTime": 1765314413801,
-  "buildNumber": 48,
-  "semanticVersion": "0.1.48",
-  "version": "0.1.48 (2025-12-09T21:06)"
+  "buildTimestamp": "2025-12-08T02:11:45.872Z",
+  "buildTime": 1765159905872,
+  "buildNumber": 38,
+  "semanticVersion": "0.1.38",
+  "version": "0.1.38 (2025-12-08T02:11)"
 }

package/codeyam-cli/templates/codeyam-setup-skill.md

@@ -170,6 +170,18 @@ codeyam update-config
 
 ### Step 2: Test Startup for Selected Webapps
 
+Ask user how they want to handle environment setup:
+
+```
+CodeYam needs to start your dev server safely without connecting to real services.
+This won't affect your normal development.
+
+Options:
+1. "My environment is already safe" - Test as-is
+2. "I'll configure it myself" - You guide, user creates mocks
+3. "Claude handles it" - You create mocks automatically
+```
+
 **IMPORTANT: You must test EVERY webapp selected in Step 1**
 
 **ACTION 1:** Get the selected webapps from config:
@@ -202,13 +214,13 @@ codeyam test-startup --webappPath . --debug
 
 - Keep a checklist of all webapps
 - Mark each as passing or failing
-- ALL selected webapps must pass before proceeding to Step 4
+- ALL webapps must pass before proceeding to Step 4
 
 **Flow for each webapp:**
 
 - If success → Move to next webapp
 - If error → Go to Step 3, create mocks, re-test THIS webapp until it passes
-- Only after ALL selected webapps pass → Go to Step 4
+- Only after ALL webapps pass → Go to Step 4
 
 ### Step 3: Fix Errors (Environment Variables or Mocks)
 
@@ -295,7 +307,7 @@ Repeat until the webapp passes, then continue testing remaining webapps.
 
 ### Step 4: Success
 
-Once `test-startup` passes for ALL selected webapps:
+Once `test-startup` passes for ALL webapps:
 
 1. Run `codeyam suggest --limit 1 --verbose` (timeout: 5 minutes)
 2. Tell user CodeYam is ready
@@ -432,7 +444,7 @@ export const prisma = {
 
 3. **Validate before retry** - Use `codeyam validate-mock` to catch issues before running test-startup again.
 
-4. **Keep mocks minimal** - Empty methods and null returns are usually enough. The goal is to provide a simple mock with a structurally accurate return value. Do not import anything for mocks. Keep the mock simple.
+4. **Keep mocks minimal** - Empty methods and null returns are usually enough.
 
 5. **Match exports exactly** - Export names are case-sensitive and must match the original.
 
@@ -443,26 +455,8 @@ export const prisma = {
 Setup is complete when:
 
 - ✅ Selected `webapps` configured in `.codeyam/config.json` with startCommand **including `$PORT` placeholder**
-- ✅ `environmentVariables` added to config (if required for startup)
+- ✅ `environmentVariables` added to config (if needed for startup)
 - ✅ `codeyam test-startup --webappPath <path> --debug` passes for all SELECTED webapps
-- ✅ User knows how to use CodeYam (analyze, test)
+- ✅ User knows how to use CodeYam (simulate, analyze, test)
 
 **CRITICAL:** The startCommand must include `$PORT` in the args (e.g., `["dev", "--port", "$PORT"]`) or env (e.g., `{"PORT": "$PORT"}`). Without this, CodeYam cannot start the server on the required port for capture.
-
-## Wrap It Up
-
-- Always end by presenting the user with a basic overview of CodeYam
-
-```
-CodeYam is set up!
-
-You can now:
-
-- Open the CodeYam dashboard: run `codeyam` or ask me to open it for you
-- Simulate any function or method in your project. Just ask me to `simulate [NAME OF FUNCTION OR METHOD]`
-- Ask me to test any function or method and I will be able to write tests more easily using the generated data structure CodeYam provides.
-
-To start how about simulating [FILE PATH AND ENTITY NAME OF SUGGESTED COMPONENT]?
-
-Do you want me to simulate that now?
-```

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codeyam/codeyam-cli",
-  "version": "0.1.0-bleeding-edge.8afd3ee",
+  "version": "0.1.0-staging.1",
   "description": "Local development CLI for CodeYam analysis",
   "type": "module",
   "bin": {

package/packages/ai/src/lib/generateChangesEntityKeyAttributes.js

@@ -93,98 +93,154 @@ export default async function generateChangesEntityKeyAttributes({ entity, exist
         llmCall,
     };
 }
-const FRONTEND_SYSTEM_MESSAGE = `You are updating the key attributes for a frontend component that was recently changed.
+const FRONTEND_SYSTEM_MESSAGE = `We have a frontend function that was recently changed.
 
-This is an INCREMENTAL update. Existing key attributes should be preserved unless the code change clearly invalidates them. Removing and re-adding attributes across changes causes unnecessary churn.
+Can you help us identify if any of the key data attributes in the function are no longer valid or if any new key data attributes should be added to the list?
 
-## Stability Rules
+We've provided a list of the existing key data attributes, the commit diff, a list of all data attributes that are now valid, and the function code.
 
-- REMOVE only if the attribute is clearly no longer used (variable deleted, conditional removed, no longer referenced)
-- KEEP marginal attributes—if unsure, preserve rather than remove
-- ADD new attributes freely when warranted
+Each key attribute represents a path from a top level variable and that path must be noted exactly as it is in the "Attributes and Value Types" list.
 
-## What Makes a Key Attribute
+Key attributes must come either from the function signature or from a function call that is made from within the function that return data that is used in the function. Please capture the attrribute path exactly from the list.
 
-A key attribute is a data path from "Attributes and Value Types" that dramatically changes how the component renders:
+If an attribute has the keyword "functionCallReturnValue" in it, it is a function call that returns data that is used in the function. That keyword simply designated the return value of the function call. If an attribute path has "[]" in it (e.g. "items[]") that represents one item in that array.
 
-- Controls a conditional that shows/hides significant UI sections
-- Changes the component's primary visual state (not just minor text differences)
-- Determines which branch of rendering logic executes
+## Key Attribute Rules
 
-NOT key attributes:
-- Values that only affect non-visible behavior (hrefs, image srcs)
-- Fallback values that won't be used if primary values exist
-- Values that would cause errors if changed
+- All key attributes must come from the "Attributes and Value Types" provided. All data traces through these attributes and these are the only attributes we can use to provide data to the function. Only use these attributes. Do not reference internally computed variables. Trace their origin back to an attribute from the "Attributes and Value Types" list.
 
-Note: A conditional that shows a different UI state (e.g., loading screen, redirect message) IS significant—include it. Only exclude guard clauses that render literally nothing.
+### Should be a key attribute
 
-## Path Format
+- Changing the value, length (e.g. for strings or arrays), etc of this variable will dramatically change how the component displays (the return value of the function).
 
-Only return attributes from the "Attributes and Value Types" list—these are the only values we can control. Use the exact path provided. Use \`functionCallReturnValue\` for function return values. Use \`[]\` for array items (e.g., \`items[]\`).
+- Is used in a conditional that dramatically alters how the component displays (note: if the conditional compares two or more variables, note which variables are involved and how they must be set to cause different behavior).
 
-## Response Format
+### Should NOT be a key attribute
 
-For \`validValueOptions\`, include boundary/extreme values that trigger different behavior—but NOT values that would cause errors.
+- Changing the value of this variable will cause an error or exception to be thrown.
+
+- Changing the value of this variable will cause a guard clause to be triggered that results in nothing being displayed.
+
+- Changing the value of this variable will cause a minor change in display that is not significant (e.g. slightly different text).
+
+- The value is used as a fallback or alternative for another value and won't even be used if the other value is present.
+
+- The value changes non-visible behavior (e.g. the href in a link or src in an image - all images sources are mocked out).
+
+## Instructions
+
+- Read each line of code. Pay particular attention to the return value of the function and how this frontend component will be displayed. What will cause it display distinctly different but functional and robust results to the user?
+- Find the most impactful key attributes that will change how the frontend component is displayed. This should be a limited subset of the "Attributes and Value Types" provided.
+- Reference the list of existing key attributes to determine if any of them are no longer valid or if any new key attributes should be added.
+- Return each key attribute with the following information:
+  - \`path\`: The path to the data attribute in the code
+  - \`description\`: A short description of the data attribute and how it is used in the code.
+  - \`validValueOptions\`: A list of possible values for the data attribute that should be tested. This should include extreme values that could cause different behavior in the code but these values should not cause an error.
+- Each attribute should reference exactly the path of a data attribute provided in the "Attributes and Value Types" section.
+
+## Response
+
+Please respond by specifying which attributes need to be added or removed. For removed attributes just specify the path that is no longer valid.
 
 \`\`\`json
 {
   "addedKeyAttributes": [
     {
-      "path": "user",
-      "description": "Controls whether the main form or redirect screen displays",
-      "validValueOptions": ["truthy user object", "null/undefined"]
-    }
+      "path": "notification.read",
+      'percentage": 50,
+      "description": "The read status of the notification determines how the message is displayed.",
+      "validValueOptions": [
+        "true",
+        "false"
+      ],
+      "errorValueOptions": []
+    },
   ],
-  "removedKeyAttributes": ["oldAttribute.path"]
+  "removedKeyAttributes": [
+    "notification.message"
+  ]
 }
 \`\`\`
 
-Return empty arrays if no changes needed.
+## Important
+
+- A key attribute must be an exact attribute path from the "Attributes and Value Types" provided. Do not return any other attribute paths.  The only data we can provide to the function comes from these attributes so any other attribute is useless. For internally computed variables trace them back to one of the higher level variables that we can actually impact. Be sure to match the text from the attribute list exactly including spaces, commas, etc.
 `;
-const BACKEND_SYSTEM_MESSAGE = `You are updating the key attributes for a backend function that was recently changed.
+const BACKEND_SYSTEM_MESSAGE = `We have a function that was recently changed.
+
+Can you help us identify if any of the key data attributes in the function are no longer valid or if any new key data attributes should be added to the list?
+
+We've provided a list of the existing key data attributes, the commit diff, a list of all data attributes that are now valid, and the function code.
 
-This is an INCREMENTAL update. Existing key attributes should be preserved unless the code change clearly invalidates them. Removing and re-adding attributes across changes causes unnecessary churn.
+Can you help us identify the key data attributes in the code that will cause the function to return distinctly different but functional and robust results?
 
-## Stability Rules
+We're not interested in causing errors or in high-level guard clauses that result in nothing being returned. We're interested in always returning a robust result that demonstrates the core intent of the function.
 
-- REMOVE only if the attribute is clearly no longer used (variable deleted, conditional removed, no longer referenced)
-- KEEP marginal attributes—if unsure, preserve rather than remove
-- ADD new attributes freely when warranted
+In order to do so we want to identify the key data attributes in the code.
 
-## What Makes a Key Attribute
+Each key attribute represents a path from a top level variable and that path must be noted exactly as it is in the "Attributes and Value Types" list.
 
-A key attribute is a data path from "Attributes and Value Types" that dramatically changes the function's behavior:
+Key attributes must come either from the function signature or from a function call that is made from within the function that return data that is used in the function. Please capture the attrribute path exactly from the list.
 
-- Controls a conditional that significantly alters execution flow or return value
-- Determines which branch of logic executes
-- For side-effect functions: changes how external systems are called (database, email, etc.)
+If an attribute has the keyword "functionCallReturnValue" in it, it is a function call that returns data that is used in the function. That keyword simply designated the return value of the function call. If an attribute path has "[]" in it (e.g. "items[]") that represents one item in that array.
 
-NOT key attributes:
-- Fallback values that won't be used if primary values exist
-- Values that would cause errors if changed
-- Guard clauses that return nothing/minimal data
+## Key Attribute Rules
 
-## Path Format
+- All key attributes must come from the "Attributes and Value Types" provided. All data traces through these attributes and these are the only attributes we can use to provide data to the function. Only use these attributes. Do not reference internally computed variables. Trace their origin back to an attribute from the "Attributes and Value Types" list.
 
-Only return attributes from the "Attributes and Value Types" list—these are the only values we can control. Use the exact path provided. Use \`functionCallReturnValue\` for function return values. Use \`[]\` for array items (e.g., \`items[]\`).
+### Should be a key attribute
 
-## Response Format
+- Changing the value, length (e.g. for strings or arrays), etc of this variable will dramatically impact the return value of the function.
 
-For \`validValueOptions\`, include boundary/extreme values that trigger different behavior—but NOT values that would cause errors.
+- Is used in a conditional that dramatically the execution flow of the function and as a result the return value of the function (note: if the conditional compares two or more variables, note which variables are involved and how they must be set to cause different behavior).
+
+- Note: some functions do not return anything but instead call out to other systems (e.g. send an email, write to a database, etc). In those cases, the key attributes should be those that dramatically change how the function interacts with those other systems.
+
+### Should NOT be a key attribute
+
+- Changing the value of this variable will cause an error or exception to be thrown.
+
+- Changing the value of this variable will cause a guard clause to be triggered that results in nothing or very little being returned.
+
+- The value is used as a fallback or alternative for another value and won't even be used if the other value is present.
+
+## Instructions
+
+- Read each line of code. Pay particular attention to the return value of the function. What data attributes will cause it to return distinctly different but functional and robust results?
+- Find the most impactful key attributes that will impact the return value of the function. This should be a limited subset of the "Attributes and Value Types" provided.
+- Reference the list of existing key attributes to determine if any of them are no longer valid or if any new key attributes should be added.
+- Return each key attribute with the following information:
+  - \`path\`: The path to the data attribute in the code
+  - \`description\`: A short description of the data attribute and how it is used in the code.
+  - \`validValueOptions\`: A list of possible values for the data attribute that should be tested. This should include extreme values that could cause different behavior in the code but these values should not cause an error.
+- Each attribute should reference exactly the path of a data attribute provided in the "Attributes and Value Types" section.
+
+## Response
+
+Please respond by specifying which attributes need to be added or removed. For removed attributes just specify the path that is no longer valid.
 
 \`\`\`json
 {
   "addedKeyAttributes": [
     {
-      "path": "config.enabled",
-      "description": "Controls whether the main processing logic runs",
-      "validValueOptions": ["true", "false"]
-    }
+      "path": "notification.read",
+      'percentage": 50,
+      "description": "The read status of the notification determines how the message is displayed.",
+      "validValueOptions": [
+        "true",
+        "false"
+      ],
+      "errorValueOptions": []
+    },
   ],
-  "removedKeyAttributes": ["oldAttribute.path"]
+  "removedKeyAttributes": [
+    "notification.message"
+  ]
 }
 \`\`\`
 
-Return empty arrays if no changes needed.
+## Important
+
+- A key attribute must be an exact attribute path from the "Attributes and Value Types" provided. Do not return any other attribute paths.  The only data we can provide to the function comes from these attributes so any other attribute is useless. For internally computed variables trace them back to one of the higher level variables that we can actually impact. Be sure to match the text from the attribute list exactly including spaces, commas, etc.
 `;
 //# sourceMappingURL=generateChangesEntityKeyAttributes.js.map

package/packages/ai/src/lib/generateChangesEntityKeyAttributes.js.map

@@ -1 +1 @@
-{"version":3,"file":"generateChangesEntityKeyAttributes.js","sourceRoot":"","sources":["../../../../../../packages/ai/src/lib/generateChangesEntityKeyAttributes.ts"],"names":[],"mappings":"AAAA,OAAO,cAAc,MAAM,kBAAkB,CAAC;AAC9C,OAAO,EAAE,WAAW,EAAE,MAAM,uBAAuB,CAAC;AAEpD,OAAO,oCAAoC,MAAM,yDAAyD,CAAC;AAC3G,OAAO,EAAE,MAAM,EAAE,MAAM,gBAAgB,CAAC;AACxC,OAAO,EAAM,oBAAoB,EAAE,MAAM,aAAa,CAAC;AACvD,OAAO,UAAU,MAAM,cAAc,CAAC;AACtC,OAAO,EAAE,mBAAmB,EAAE,MAAM,wCAAwC,CAAC;AAU7E,MAAM,CAAC,OAAO,CAAC,KAAK,UAAU,kCAAkC,CAAC,EAC/D,MAAM,EACN,qBAAqB,EACrB,QAAQ,EACR,UAAU,EACV,KAAK,GACkC;IACvC,MAAM,CAAC,2CAA2C,EAAE;QAClD,QAAQ,EAAE,MAAM,CAAC,QAAQ;QACzB,UAAU,EAAE,MAAM,CAAC,IAAI;QACvB,UAAU,EAAE,QAAQ,CAAC,EAAE;QACvB,qBAAqB,EAAE,qBAAqB,EAAE,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,YAAY,CAAC;KACzE,CAAC,CAAC;IAEH,MAAM,EAAE,aAAa,EAAE,cAAc,EAAE,oBAAoB,EAAE,GAC3D,mBAAmB,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;IAEpC,MAAM,MAAM,GAAG,oCAAoC,CAAC;QAClD,IAAI,EAAE,MAAM,CAAC,IAAI;QACjB,aAAa;QACb,qBAAqB;QACrB,UAAU;KACX,CAAC,CAAC;IAEH,MAAM,QAAQ,GAAG,MAAM,cAAc,CAAC;QACpC,IAAI,EAAE,oCAAoC;QAC1C,aAAa,EAAE,UAAU,CAAC,MAAM,CAAC,QAAQ,CAAC;YACxC,CAAC,CAAC,uBAAuB;YACzB,CAAC,CAAC,sBAAsB;QAC1B,MAAM;QACN,KAAK,EAAE,KAAK,IAAI,oBAAoB;KACrC,CAAC,CAAC;IAEH,MAAM,OAAO,GAAG,MAAM,WAAW,CAAC;QAChC,WAAW,EAAE,UAAU;QACvB,SAAS,EAAE,QAAQ,CAAC,EAAE;QACtB,SAAS,EAAE;YACT,MAAM,EAAE;gBACN,IAAI,EAAE,MAAM,CAAC,IAAI;gBACjB,QAAQ,EAAE,MAAM,CAAC,QAAQ;gBACzB,IAAI,EAAE,MAAM,CAAC,IAAI;aAClB;YACD,QAAQ,EAAE;gBACR,EAAE,EAAE,QAAQ,CAAC,EAAE;gBACf,QAAQ,EAAE;oBACR,qBAAqB,EAAE,MAAM,CAAC,QAAQ,CAAC,qBAAqB;oBAC5D,aAAa,EAAE,QAAQ,CAAC,QAAQ,EAAE,aAAa,IAAI,EAAE;iBACtD;aACF;YACD,UAAU;YACV,KAAK;SACN;QACD,GAAG,QAAQ,CAAC,KAAK;KAClB,CAAC,CAAC;IAEH,MAAM,EAAE,UAAU,EAAE,mBAAmB,EAAE,GAAG,QAAQ,CAAC;IACrD,IAAI,CAAC,mBAAmB,EAAE,CAAC;QACzB,OAAO,CAAC,GAAG,CACT,2FAA2F,CAC5F,CAAC;QACF,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,CACJ,WAAW,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,MAAM,CAAC,QAAQ,IAAI,MAAM,CAAC,IAAI,8BAA8B,EACxG,mBAAmB,CACpB,CAAC;IAEF,MAAM,EAAE,kBAAkB,EAAE,oBAAoB,EAAE,GAChD,IAAI,CAAC,KAAK,CAAC,mBAAmB,CAAC,CAAC;IAElC,MAAM,aAAa,GAA0C;QAC3D,GAAG,CAAC,qBAAqB,IAAI,EAAE,CAAC;KACjC,CAAC;IAEF,KAAK,MAAM,iBAAiB,IAAI,kBAAkB,IAAI,EAAE,EAAE,CAAC;QACzD,MAAM,YAAY,GAAG,iBAAiB,CAAC,IAAI,CAAC;QAC5C,MAAM,KAAK,GAAG,aAAa,CAAC,SAAS,CACnC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,YAAY,KAAK,YAAY,CACvC,CAAC;QACF,MAAM,gBAAgB,GAAG;YACvB,YAAY;YACZ,YAAY,EAAE,cAAc,CAAC,YAAY,CAAC;YAC1C,iBAAiB,EAAE,oBAAoB,CAAC,YAAY,CAAC,IAAI,YAAY;YACrE,WAAW,EAAE,iBAAiB,CAAC,WAAW;YAC1C,iBAAiB,EAAE,iBAAiB,CAAC,iBAAiB;YACtD,iBAAiB,EAAE,iBAAiB,CAAC,iBAAiB;SACvD,CAAC;QACF,IAAI,KAAK,KAAK,CAAC,CAAC,EAAE,CAAC;YACjB,aAAa,CAAC,MAAM,CAAC,KAAK,EAAE,CAAC,EAAE,gBAAgB,CAAC,CAAC;QACnD,CAAC;aAAM,CAAC;YACN,aAAa,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;QACvC,CAAC;IACH,CAAC;IAED,KAAK,MAAM,mBAAmB,IAAI,oBAAoB,IAAI,EAAE,EAAE,CAAC;QAC7D,MAAM,YAAY,GAChB,OAAO,mBAAmB,KAAK,QAAQ;YACrC,CAAC,CAAC,mBAAmB;YACrB,CAAC,CAAC,mBAAmB,CAAC,IAAI,CAAC;QAC/B,MAAM,KAAK,GAAG,aAAa,CAAC,SAAS,CACnC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,YAAY,KAAK,YAAY,CACvC,CAAC;QACF,IAAI,KAAK,KAAK,CAAC,CAAC,EAAE,CAAC;YACjB,aAAa,CAAC,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC;QACjC,CAAC;IACH,CAAC;IAED,MAAM,yBAAyB,GAAG,aAAa;SAC5C,MAAM,CACL,CAAC,YAAY,EAAE,KAAK,EAAE,IAAI,EAAE,EAAE,CAC5B,IAAI,CAAC,SAAS,CACZ,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,YAAY,KAAK,YAAY,CAAC,YAAY,CACtD,KAAK,KAAK,CACd;SACA,MAAM,CAAC,CAAC,YAAY,EAAE,EAAE,CAAC,aAAa,CAAC,YAAY,CAAC,YAAY,CAAC,CAAC,CAAC;IAEtE,OAAO;QACL,aAAa,EAAE,yBAAyB;QACxC,OAAO;KACR,CAAC;AACJ,CAAC;AAED,MAAM,uBAAuB,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA+C/B,CAAC;AAEF,MAAM,sBAAsB,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA6C9B,CAAC"}
+{"version":3,"file":"generateChangesEntityKeyAttributes.js","sourceRoot":"","sources":["../../../../../../packages/ai/src/lib/generateChangesEntityKeyAttributes.ts"],"names":[],"mappings":"AAAA,OAAO,cAAc,MAAM,kBAAkB,CAAC;AAC9C,OAAO,EAAE,WAAW,EAAE,MAAM,uBAAuB,CAAC;AAEpD,OAAO,oCAAoC,MAAM,yDAAyD,CAAC;AAC3G,OAAO,EAAE,MAAM,EAAE,MAAM,gBAAgB,CAAC;AACxC,OAAO,EAAM,oBAAoB,EAAE,MAAM,aAAa,CAAC;AACvD,OAAO,UAAU,MAAM,cAAc,CAAC;AACtC,OAAO,EAAE,mBAAmB,EAAE,MAAM,wCAAwC,CAAC;AAU7E,MAAM,CAAC,OAAO,CAAC,KAAK,UAAU,kCAAkC,CAAC,EAC/D,MAAM,EACN,qBAAqB,EACrB,QAAQ,EACR,UAAU,EACV,KAAK,GACkC;IACvC,MAAM,CAAC,2CAA2C,EAAE;QAClD,QAAQ,EAAE,MAAM,CAAC,QAAQ;QACzB,UAAU,EAAE,MAAM,CAAC,IAAI;QACvB,UAAU,EAAE,QAAQ,CAAC,EAAE;QACvB,qBAAqB,EAAE,qBAAqB,EAAE,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,YAAY,CAAC;KACzE,CAAC,CAAC;IAEH,MAAM,EAAE,aAAa,EAAE,cAAc,EAAE,oBAAoB,EAAE,GAC3D,mBAAmB,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;IAEpC,MAAM,MAAM,GAAG,oCAAoC,CAAC;QAClD,IAAI,EAAE,MAAM,CAAC,IAAI;QACjB,aAAa;QACb,qBAAqB;QACrB,UAAU;KACX,CAAC,CAAC;IAEH,MAAM,QAAQ,GAAG,MAAM,cAAc,CAAC;QACpC,IAAI,EAAE,oCAAoC;QAC1C,aAAa,EAAE,UAAU,CAAC,MAAM,CAAC,QAAQ,CAAC;YACxC,CAAC,CAAC,uBAAuB;YACzB,CAAC,CAAC,sBAAsB;QAC1B,MAAM;QACN,KAAK,EAAE,KAAK,IAAI,oBAAoB;KACrC,CAAC,CAAC;IAEH,MAAM,OAAO,GAAG,MAAM,WAAW,CAAC;QAChC,WAAW,EAAE,UAAU;QACvB,SAAS,EAAE,QAAQ,CAAC,EAAE;QACtB,SAAS,EAAE;YACT,MAAM,EAAE;gBACN,IAAI,EAAE,MAAM,CAAC,IAAI;gBACjB,QAAQ,EAAE,MAAM,CAAC,QAAQ;gBACzB,IAAI,EAAE,MAAM,CAAC,IAAI;aAClB;YACD,QAAQ,EAAE;gBACR,EAAE,EAAE,QAAQ,CAAC,EAAE;gBACf,QAAQ,EAAE;oBACR,qBAAqB,EAAE,MAAM,CAAC,QAAQ,CAAC,qBAAqB;oBAC5D,aAAa,EAAE,QAAQ,CAAC,QAAQ,EAAE,aAAa,IAAI,EAAE;iBACtD;aACF;YACD,UAAU;YACV,KAAK;SACN;QACD,GAAG,QAAQ,CAAC,KAAK;KAClB,CAAC,CAAC;IAEH,MAAM,EAAE,UAAU,EAAE,mBAAmB,EAAE,GAAG,QAAQ,CAAC;IACrD,IAAI,CAAC,mBAAmB,EAAE,CAAC;QACzB,OAAO,CAAC,GAAG,CACT,2FAA2F,CAC5F,CAAC;QACF,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,CACJ,WAAW,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,MAAM,CAAC,QAAQ,IAAI,MAAM,CAAC,IAAI,8BAA8B,EACxG,mBAAmB,CACpB,CAAC;IAEF,MAAM,EAAE,kBAAkB,EAAE,oBAAoB,EAAE,GAChD,IAAI,CAAC,KAAK,CAAC,mBAAmB,CAAC,CAAC;IAElC,MAAM,aAAa,GAA0C;QAC3D,GAAG,CAAC,qBAAqB,IAAI,EAAE,CAAC;KACjC,CAAC;IAEF,KAAK,MAAM,iBAAiB,IAAI,kBAAkB,IAAI,EAAE,EAAE,CAAC;QACzD,MAAM,YAAY,GAAG,iBAAiB,CAAC,IAAI,CAAC;QAC5C,MAAM,KAAK,GAAG,aAAa,CAAC,SAAS,CACnC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,YAAY,KAAK,YAAY,CACvC,CAAC;QACF,MAAM,gBAAgB,GAAG;YACvB,YAAY;YACZ,YAAY,EAAE,cAAc,CAAC,YAAY,CAAC;YAC1C,iBAAiB,EAAE,oBAAoB,CAAC,YAAY,CAAC,IAAI,YAAY;YACrE,WAAW,EAAE,iBAAiB,CAAC,WAAW;YAC1C,iBAAiB,EAAE,iBAAiB,CAAC,iBAAiB;YACtD,iBAAiB,EAAE,iBAAiB,CAAC,iBAAiB;SACvD,CAAC;QACF,IAAI,KAAK,KAAK,CAAC,CAAC,EAAE,CAAC;YACjB,aAAa,CAAC,MAAM,CAAC,KAAK,EAAE,CAAC,EAAE,gBAAgB,CAAC,CAAC;QACnD,CAAC;aAAM,CAAC;YACN,aAAa,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;QACvC,CAAC;IACH,CAAC;IAED,KAAK,MAAM,mBAAmB,IAAI,oBAAoB,IAAI,EAAE,EAAE,CAAC;QAC7D,MAAM,YAAY,GAChB,OAAO,mBAAmB,KAAK,QAAQ;YACrC,CAAC,CAAC,mBAAmB;YACrB,CAAC,CAAC,mBAAmB,CAAC,IAAI,CAAC;QAC/B,MAAM,KAAK,GAAG,aAAa,CAAC,SAAS,CACnC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,YAAY,KAAK,YAAY,CACvC,CAAC;QACF,IAAI,KAAK,KAAK,CAAC,CAAC,EAAE,CAAC;YACjB,aAAa,CAAC,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC;QACjC,CAAC;IACH,CAAC;IAED,MAAM,yBAAyB,GAAG,aAAa;SAC5C,MAAM,CACL,CAAC,YAAY,EAAE,KAAK,EAAE,IAAI,EAAE,EAAE,CAC5B,IAAI,CAAC,SAAS,CACZ,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,YAAY,KAAK,YAAY,CAAC,YAAY,CACtD,KAAK,KAAK,CACd;SACA,MAAM,CAAC,CAAC,YAAY,EAAE,EAAE,CAAC,aAAa,CAAC,YAAY,CAAC,YAAY,CAAC,CAAC,CAAC;IAEtE,OAAO;QACL,aAAa,EAAE,yBAAyB;QACxC,OAAO;KACR,CAAC;AACJ,CAAC;AAED,MAAM,uBAAuB,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAwE/B,CAAC;AAEF,MAAM,sBAAsB,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA4E9B,CAAC"}

package/packages/ai/src/lib/generateEntityKeyAttributes.js

@@ -106,77 +106,224 @@ export default async function generateEntityKeyAttributes({ entity, model, }) {
         llmCall,
     };
 }
-const FRONTEND_SYSTEM_MESSAGE = `You analyze frontend components to identify key data attributes that significantly impact rendering.
+const FRONTEND_SYSTEM_MESSAGE = `We have a frontend function and need to determine how best to vary what is returned from this function and displayed to the user.
 
-## What Makes a Key Attribute
+We're not interested in causing errors or in high-level guard clauses that often display nothing on the screen. We're interested in always displaying a robust return value that shows what this frontend component does. This does not mean that values can't be undefined, but only allow a "validValueOption" to be "undefined" if the component will still display something meaningful to the user. 
 
-A key attribute is a data path from "Attributes and Value Types" that dramatically changes how the component renders:
+In order to do so we want to identify the key data attributes in the code. We've included a list of attributes we've identified as being used in conditionals in the code. Can you start with those, order them by importance and remove any that will result in nothing being displayed?
 
-- Controls a conditional that shows/hides significant UI sections
-- Changes the component's primary visual state (not just minor text differences)
-- Determines which branch of rendering logic executes
+Then can you idfentify any other key attributes from the "Attributes and Value Types" list that will dramatically change how the component is displayed to the user?
 
-NOT key attributes:
-- Values that only affect non-visible behavior (hrefs, image srcs)
-- Fallback values that won't be used if primary values exist
-- Values that would cause errors if changed
-- Guard clauses that render literally nothing
+Each key attribute represents a path from a top level variable and that path must be noted exactly as it is in the "Attributes and Value Types" list.
 
-Note: A conditional that shows a different UI state (e.g., loading screen, redirect message) IS significant—include it.
+Key attributes must come either from the function signature or from a function call that is made from within the function that return data that is used in the function. Please capture the attrribute path exactly from the list.
 
-## Path Format
+If an attribute has the keyword "functionCallReturnValue" in it, it is a function call that returns data that is used in the function. That keyword simply designated the return value of the function call. If an attribute path has "[]" in it (e.g. "items[]") that represents one item in that array.
 
-Only return attributes from the "Attributes and Value Types" list—these are the only values we can control. Use the exact \`fullPath\` provided. Use \`functionCallReturnValue\` for function return values. Use \`[]\` for array items (e.g., \`items[]\`).
+## Key Attribute Rules
 
-## Response Format
+- All key attributes must come from the "Attributes and Value Types" provided. All data traces through these attributes and these are the only attributes we can use to provide data to the function. Only use these attributes. Do not reference internally computed variables. Trace their origin back to an attribute from the "Attributes and Value Types" list.
 
-For \`validValueOptions\`, include boundary/extreme values that trigger different behavior—but NOT values that would cause errors.
+### Should be a key attribute
+
+- Changing the presence, value, length (e.g. for strings or arrays), etc of this variable will dramatically change how the component displays (the return value of the function).
+
+- Is used in a conditional (especially conditionals in the JSX such as \`{varName1 && varName2 && ([JSX])}\` where the absence or presence of both \`varName1\` and \`varName2\` directly impact what the user sees) that alters how the component displays. Capture how the conditional uses the variable. Does it require the variable to be true/false, defined/not defined (e.g. undefined), greater than/less than a certain value, compared with another variable, etc. Really any attribute that is used in a conditional should be a key attribute.
+
+### Should NOT be a key attribute
+
+- Changing the value of this variable will cause an error or exception to be thrown.
+
+- Changing the value of this variable will cause a guard clause around the whole method or function to be triggered that results in nothing at all being displayed.
+
+- Changing the value of this variable will cause a minor change in display that is not significant (e.g. slightly different text).
+
+- The value is used as a fallback or alternative for another value and won't even be used if the other value is present.
+
+- The value changes non-visible behavior (e.g. the href in a link or src in an image - all images sources are mocked out).
+
+## Instructions
+
+- Read each line of code. Pay particular attention to the return value of the function and how this frontend component will be displayed. What will cause it display distinctly different but functional and robust results to the user?
+- Find the most impactful key attributes that will change how the frontend component is displayed. This should be a limited subset of the "Attributes and Value Types" provided.
+- Return each key attribute with the following information:
+  - \`fullPath\`: The fullPath to the data attribute in the code
+  - \`description\`: A short description of the data attribute and how it is used in the code.
+  - \`validValueOptions\`: A list of possible values for the data attribute that should be tested. This list should be minimal and only include values that the code will handle properly and will clearly make a difference in how the component is displayed. Do not make up arbitrary values without clear evidence that each different value will cause a different display behavior.
+- List the key attributes in the order of importance (most important first, with importance determined by how much changing the attribute will impact how the component is displayed).
+- Each attribute should reference exactly the full fullPath of a data attribute provided in the "Attributes and Value Types" section.
+
+## Example
+
+Given the following "Attributes and Value Types":
+\`\`\`
+  'notification.read': { fullPath: 'signature[0].notification.read', type: 'boolean' }
+  'notification.message': { fullPath: 'signature[0].notification.message', type: 'string' }
+\`\`\`
+
+and code:
+
+\`\`\`typescript
+const NotificationText = (notification) => {
+  if (notification.read) {
+    return (
+      <span onClick={markAsRead(notification.id)}>
+        <img src={notification.icon} alt="Notification Icon" />
+        {notification.message}
+      </span>
+    );
+  } else {
+    return <strong>{notification.message}</strong>;
+  }
+};
+\`\`\`
+
+The response might be:
 
 \`\`\`json
 {
   "keyAttributes": [
     {
-      "fullPath": "signature[0].user",
-      "description": "Controls whether the main form or redirect screen displays",
-      "validValueOptions": ["truthy user object", "null/undefined"]
+      "fullPath": "signature[0].notification.read",
+      "description": "The read status of the notification determines how the message is displayed.",
+      "validValueOptions": [
+        "true",
+        "false"
+      ]
+    },
+    {
+      "fullPath": "signature[0].notification.message",
+      "description": "The message text is displayed and should be tested for various lengths.",
+      "validValueOptions": [
+        "An empty message (no characters)",
+        "A medium message (around 100 characters)",
+        "A long message (longer than 200 characters)"
+      ]
     }
   ]
 }
 \`\`\`
+
+## Important
+
+- A key attribute must be an exact attribute fullPath from the "Attributes and Value Types" provided. Do not return any other attribute fullPaths.  The only data we can provide to the function comes from these attributes so any other attribute is useless. For internally computed variables trace them back to one of the higher level variables that we can actually impact. Be sure to match the text from the attribute list exactly including spaces, commas, etc.
 `;
-const BACKEND_SYSTEM_MESSAGE = `You analyze backend functions to identify key data attributes that significantly impact behavior.
+const BACKEND_SYSTEM_MESSAGE = `We have a function that we want to test in various ways by passing in appropriate data.
+
+Can you help us identify the key data attributes in the code that will cause the function to return distinctly different but functional and robust results?
+
+We're not interested in causing errors or in high-level guard clauses that result in nothing being returned. We're interested in always returning a robust result that demonstrates the core intent of the function.
 
-## What Makes a Key Attribute
+In order to do so we want to identify the key data attributes in the code. We've included a list of attributes we've identified as being used in conditionals in the code. Can you start with those, order them by importance and remove any that will result in nothing being displayed?
 
-A key attribute is a data path from "Attributes and Value Types" that dramatically changes the function's behavior:
+Then can you idfentify any other key attributes from the "Attributes and Value Types" list that will dramatically change how the component is displayed to the user?
 
-- Controls a conditional that significantly alters execution flow or return value
-- Determines which branch of logic executes
-- For side-effect functions: changes how external systems are called (database, email, etc.)
+Each key attribute represents a path from a top level variable and that fullPath must be noted exactly as it is in the "Attributes and Value Types" list.
 
-NOT key attributes:
-- Fallback values that won't be used if primary values exist
-- Values that would cause errors if changed
-- Guard clauses that return nothing/minimal data
+Key attributes must come either from the function signature or from a function call that is made from within the function that return data that is used in the function. Please capture the attrribute fullPath exactly from the list.
 
-## Path Format
+If an attribute has the keyword "functionCallReturnValue" in it, it is a function call that returns data that is used in the function. That keyword simply designated the return value of the function call. If an attribute path has "[]" in it (e.g. "items[]") that represents one item in that array.
 
-Only return attributes from the "Attributes and Value Types" list—these are the only values we can control. Use the exact \`fullPath\` provided. Use \`functionCallReturnValue\` for function return values. Use \`[]\` for array items (e.g., \`items[]\`).
+## Key Attribute Rules
 
-## Response Format
+- All key attributes must come from the "Attributes and Value Types" provided. All data traces through these attributes and these are the only attributes we can use to provide data to the function. Only use these attributes. Do not reference internally computed variables. Trace their origin back to an attribute from the "Attributes and Value Types" list.
 
-For \`validValueOptions\`, include boundary/extreme values that trigger different behavior—but NOT values that would cause errors.
+### Should be a key attribute
+
+- Changing the value, length (e.g. for strings or arrays), etc of this variable will dramatically impact the return value of the function.
+
+- Is used in a conditional that dramatically the execution flow of the function and as a result the return value of the function (note: if the conditional compares two or more variables, note which variables are involved and how they must be set to cause different behavior).
+
+- Note: some functions do not return anything but instead call out to other systems (e.g. send an email, write to a database, etc). In those cases, the key attributes should be those that dramatically change how the function interacts with those other systems.
+
+### Should NOT be a key attribute
+
+- Changing the value of this variable will cause an error or exception to be thrown.
+
+- Changing the value of this variable will cause a guard clause to be triggered that results in nothing or very little being returned.
+
+- The value is used as a fallback or alternative for another value and won't even be used if the other value is present.
+
+## Instructions
+
+- Read each line of code. Pay particular attention to the return value of the function. What data attributes will cause it to return distinctly different but functional and robust results?
+- Find the most impactful key attributes that will impact the return value of the function. This should be a limited subset of the "Attributes and Value Types" provided.
+- Return each key attribute with the following information:
+  - \`fullPath\`: The fullPath to the data attribute in the code
+  - \`description\`: A short description of the data attribute and how it is used in the code.
+  - \`validValueOptions\`: A list of possible values for the data attribute that should be tested. This list should be minimal and only include values that the code will handle properly and will clearly make a difference in the functions's behavior. Do not make up arbitrary values without clear evidence that each different value will clearly result in a different function behavior.
+- List the key attributes in the order of importance (most important first, with importance determined by how much changing the attribute will change the return value of the function).
+- Each attribute should reference exactly the fullPath of a data attribute provided in the "Attributes and Value Types" section.
+
+## Example
+
+Given the following "Attributes and Value Types":
+\`\`\`
+[
+  items[].quantity: { fullPath: 'signature[0][].quantity', type: 'number' },
+  items[].price: { fullPath: 'signature[0][].price', type: 'number' },
+  discount: { fullPath: 'signature[1]', type: 'number' },
+  taxRate: { fullPath: 'signature[2]', type: 'number' },
+]
+\`\`\`
+
+and code:
+
+\`\`\`typescript
+const calculateTotal = (items, discount, taxRate) => {
+  const subtotal = items.reduce((sum, item) => sum + item.quantity * item.price, 0);
+  const discountedTotal = subtotal - discount;
+  const tax = discountedTotal * taxRate;
+  return discountedTotal + tax;
+}
+\`\`\`
+
+The response might be:
 
 \`\`\`json
 {
   "keyAttributes": [
     {
-      "fullPath": "signature[0].config.enabled",
-      "description": "Controls whether the main processing logic runs",
-      "validValueOptions": ["true", "false"]
+      "fullPath": "signature[0][].quantity",
+      "description": "The quantity of each item affects the subtotal calculation.",
+      "validValueOptions": [
+        "0 (no items)",
+        "1 (single item)",
+        "5 (multiple items)",
+        "100 (large quantity)"
+      ]
+    },
+    {
+      "fullPath": "signature[0][].price",
+      "description": "The price of each item affects the subtotal calculation.",
+      "validValueOptions": [
+        "0.00 (free item)",
+        "19.99 (average price)",
+        "999.99 (high price)"
+      ]
+    },
+    {
+      "fullPath": "signature[1]",
+      "description": "The discount amount directly reduces the subtotal.",
+      "validValueOptions": [
+        "0 (no discount)",
+        "20.00 (discount)"
+      ]
+    },
+    {
+      "fullPath": "signature[2]",
+      "description": "The tax rate affects the final total calculation.",
+      "validValueOptions": [
+        "0 (no tax)",
+        "0.20 (20% tax)"
+      ]
     }
   ]
 }
 \`\`\`
+
+## Important
+
+- A key attribute must be an exact attribute fullPath from the "Attributes and Value Types" provided. Do not return any other attribute fullPaths.  The only data we can provide to the function comes from these attributes so any other attribute is useless. For internally computed variables trace them back to one of the higher level variables that we can actually impact. Be sure to match the text from the attribute list exactly including spaces, commas, etc.
 `;
 //# sourceMappingURL=generateEntityKeyAttributes.js.map