Merge branch 'main' into jl-rate-history-flaky-test-fix

.github/workflows/deploy.yml

@@ -211,7 +211,7 @@ jobs:
       - name: Check out repository
         uses: actions/checkout@v4
 
-      - uses: actions/setup-node@v3
+      - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
           cache: yarn

.github/workflows/promote.yml

@@ -81,7 +81,7 @@ jobs:
       - name: Check out repository
         uses: actions/checkout@v4
 
-      - uses: actions/setup-node@v3
+      - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
           cache: 'yarn'

docs/technical-design/howto-update-state-programs.md

@@ -11,6 +11,7 @@ The source of truth for that file comes from a CSV maintained by product and des
 
 1. Download the latest version of csv from google docs when prompted by product/design.
 2. Run the script following the command listed in the  `import-programs.ts`.
-3. Overwrite existing state programs JSON with the new output. Your usage of the script will likely look something like this:  `cd scripts && yarn tsc && node import-programs.js ~/Desktop/State\ programs,\ population,\ and\ nicknames.csv > ../services/app-web/src/common-code/data/statePrograms.json`
+3. Overwrite existing state programs JSON with the new output. Your usage of the script will likely look something like this:  `cd scripts && yarn tsc && node import-programs.js path/to/data.csv > ../services/app-web/src/common-code/data/statePrograms.json`
 4. Double check the diff. It's important not to delete any programs that have already been used for a submission because although programs are not in the database, we still store references to the program ID in postgres as if they are stable. Also, we want to be sure we are only changing programs expected to change.
-5. Make a PR to update the statePrograms file in the codebase
+5. For any newly created programs, manually populate the `id` field usings a UUID generator 
+6. Make a PR to update the statePrograms file in the codebase

scripts/import-programs.ts

@@ -2,13 +2,16 @@
 This script is used to generate a list of MC-Review state programs. To read more about this script and why it's used see "How to update state programs" technical design docs.
 
 To run:
- yarn tsc && node ./import-programs.js path/to/data.csv
+ yarn tsc && node ./import-programs.js path/to/data.csv > ../services/app-web/src/common-code/data/statePrograms.json
 
 The input file is expected to be a valid CSV with at least the following columns:
    1 State (two-character state code, uppercase)
    2 Program (full program name)
    3 Nickname (acronym or abbreviation e.g. "CME")
 
+Documentation for this script can be found here:
+https://github.com/Enterprise-CMCS/managed-care-review/blob/main/docs/technical-design/howto-update-state-programs.md
+
  Additional columns aren't used and should be ignored.
  */
 

services/app-api/src/resolvers/attributeHelper.ts

@@ -8,7 +8,10 @@ export function setResolverDetailsOnActiveSpan(
     user: Context['user'],
     span: Context['span']
 ): void {
-    if (!span) return
+    if (!span) {
+        console.info(`No span set on ${name} call`)
+        return
+    }
     span.setAttributes({
         [SemanticAttributes.ENDUSER_ID]: user.email,
         [SemanticAttributes.ENDUSER_ROLE]: user.role,

services/app-api/src/resolvers/healthPlanPackage/indexHealthPlanPackages.ts

@@ -53,7 +53,8 @@ export function indexHealthPlanPackagesResolver(
 ): QueryResolvers['indexHealthPlanPackages'] {
     return async (_parent, _args, context) => {
         const { user, span } = context
-        setResolverDetailsOnActiveSpan('fetchHealthPlanPackage', user, span)
+
+        setResolverDetailsOnActiveSpan('indexHealthPlanPackages', user, span)
 
         const ratesDatabaseRefactor = await launchDarkly.getFeatureFlag(
             context,
@@ -70,7 +71,7 @@ export function indexHealthPlanPackagesResolver(
 
                 if (contractsWithHistory instanceof Error) {
                     const errMessage = `Issue finding contracts with history by stateCode: ${user.stateCode}. Message: ${contractsWithHistory.message}`
-                    logError('fetchHealthPlanPackage', errMessage)
+                    logError('indexHealthPlanPackages', errMessage)
                     setErrorAttributesOnActiveSpan(errMessage, span)
 
                     if (contractsWithHistory instanceof NotFoundError) {
@@ -111,7 +112,7 @@ export function indexHealthPlanPackagesResolver(
 
                 if (contractsWithHistory instanceof Error) {
                     const errMessage = `Issue finding contracts with history by submit info. Message: ${contractsWithHistory.message}`
-                    logError('fetchHealthPlanPackage', errMessage)
+                    logError('indexHealthPlanPackages', errMessage)
                     setErrorAttributesOnActiveSpan(errMessage, span)
 
                     if (contractsWithHistory instanceof NotFoundError) {

services/app-api/src/resolvers/user/fetchCurrentUser.test.ts

@@ -16,7 +16,7 @@ describe('currentUser', () => {
 
         expect(res.data?.fetchCurrentUser.email).toBe('james@example.com')
         expect(res.data?.fetchCurrentUser.state.code).toBe('FL')
-        expect(res.data?.fetchCurrentUser.state.programs).toHaveLength(6)
+        expect(res.data?.fetchCurrentUser.state.programs).toHaveLength(5)
     })
 
     it('returns programs for MI', async () => {

services/app-proto/src/health_plan_form_data.proto

@@ -315,4 +315,5 @@ enum StateCode {
   STATE_CODE_WI = 50;
   STATE_CODE_WV = 51;
   STATE_CODE_WY= 52;
+  STATE_CODE_KY = 53;
 }