feat: add workspace support

Author: sorccu

packages/cli/src/commands/baseCommand.ts

@@ -3,8 +3,8 @@ import prompts from 'prompts'
 import { Command } from '@oclif/core'
 import { api } from '../rest/api'
 import { CommandStyle } from '../helpers/command-style'
-import { PackageFilesResolver } from '../services/check-parser/package-files/resolver'
 import { PackageJsonFile } from '../services/check-parser/package-files/package-json-file'
+import { detectNearestPackageJson } from '../services/check-parser/package-files/package-manager'
 
 export type BaseCommandClass = typeof Command & {
   coreCommand: boolean
@@ -18,12 +18,15 @@ export abstract class BaseCommand extends Command {
   #packageJsonLoader?: Promise<PackageJsonFile | undefined>
 
   async loadPackageJsonOfSelf (): Promise<PackageJsonFile | undefined> {
-    if (!this.#packageJsonLoader) {
-      const resolver = new PackageFilesResolver()
-      this.#packageJsonLoader = resolver.loadPackageJsonFile(__filename)
-    }
+    try {
+      if (!this.#packageJsonLoader) {
+        this.#packageJsonLoader = detectNearestPackageJson(__dirname)
+      }
 
-    return await this.#packageJsonLoader
+      return await this.#packageJsonLoader
+    } catch {
+      return
+    }
   }
 
   async checkEngineCompatibility (): Promise<void> {

packages/cli/src/commands/import/plan.ts

@@ -26,9 +26,8 @@ import { ExitError } from '@oclif/core/errors'
 import { confirmCommit, performCommitAction } from './commit'
 import { confirmApply, performApplyAction } from './apply'
 import { generateChecklyConfig } from '../../services/checkly-config-codegen'
-import { PackageFilesResolver } from '../../services/check-parser/package-files/resolver'
 import { PackageJsonFile } from '../../services/check-parser/package-files/package-json-file'
-import { detectPackageManager, knownPackageManagers, PackageManager } from '../../services/check-parser/package-files/package-manager'
+import { detectNearestPackageJson, detectPackageManager, knownPackageManagers, PackageManager } from '../../services/check-parser/package-files/package-manager'
 import { parseProject } from '../../services/project-parser'
 import { Runtime } from '../../rest/runtimes'
 import { ConstructExport, Project, Session } from '../../constructs/project'
@@ -704,10 +703,11 @@ ${chalk.cyan('For safety, resources are not deletable until the plan has been co
   }
 
   async #loadPackageJson (): Promise<PackageJsonFile | undefined> {
-    const resolver = new PackageFilesResolver()
-    return await resolver.loadPackageJsonFile(process.cwd(), {
-      isDir: true,
-    })
+    try {
+      return await detectNearestPackageJson(process.cwd())
+    } catch {
+      return
+    }
   }
 
   #createPackageJson (logicalId: string): PackageJsonFile {

packages/cli/src/constructs/browser-check.ts

@@ -1,9 +1,7 @@
 import fs from 'node:fs/promises'
-import path from 'node:path'
 
 import { CheckProps, RuntimeCheck, RuntimeCheckProps } from './check'
 import { Session, SharedFileRef } from './project'
-import { pathToPosix } from '../services/util'
 import { Content, Entrypoint, isContent, isEntrypoint } from './construct'
 import { detectSnapshots } from '../services/snapshot-service'
 import { PlaywrightConfig } from './playwright-config'
@@ -168,7 +166,7 @@ export class BrowserCheck extends RuntimeCheck {
     const deps: SharedFileRef[] = []
     for (const { filePath, content } of parsed.dependencies) {
       deps.push(Session.registerSharedFile({
-        path: pathToPosix(path.relative(Session.basePath!, filePath)),
+        path: Session.relativePosixPath(filePath),
         content,
       }))
     }

packages/cli/src/constructs/check-group-v1.ts

@@ -17,7 +17,6 @@ import { Diagnostics } from './diagnostics'
 import { DeprecatedConstructDiagnostic, DeprecatedPropertyDiagnostic, InvalidPropertyValueDiagnostic } from './construct-diagnostics'
 import CheckTypes from '../constants'
 import { CheckConfigDefaults } from '../services/checkly-config-loader'
-import { pathToPosix } from '../services/util'
 import { AlertChannelSubscription } from './alert-channel-subscription'
 import { BrowserCheck } from './browser-check'
 import { CheckGroupRef } from './check-group-ref'
@@ -438,7 +437,7 @@ export class CheckGroupV1 extends Construct {
         },
         // the browserChecks props inherited from the group are applied in BrowserCheck.constructor()
       }
-      const checkLogicalId = pathToPosix(path.relative(Session.basePath!, filepath))
+      const checkLogicalId = Session.relativePosixPath(filepath)
       if (checkType === CheckTypes.BROWSER) {
         new BrowserCheck(checkLogicalId, props)
       } else {

packages/cli/src/constructs/internal/codegen/snippet.ts

@@ -54,6 +54,6 @@ export function parseSnippetDependencies (content: string): string[] {
   }
 
   return dependencies
-    .filter(value => value.startsWith(SNIPPET_PATH_PREFIX))
-    .map(value => value.slice(SNIPPET_PATH_PREFIX.length))
+    .filter(({ importPath }) => importPath.startsWith(SNIPPET_PATH_PREFIX))
+    .map(({ importPath }) => importPath.slice(SNIPPET_PATH_PREFIX.length))
 }

packages/cli/src/constructs/project.ts

@@ -24,6 +24,7 @@ import { Diagnostics } from './diagnostics'
 import { ConstructDiagnostics, InvalidPropertyValueDiagnostic } from './construct-diagnostics'
 import { ProjectBundle, ProjectDataBundle } from './project-bundle'
 import { pathToPosix } from '../services/util'
+import { Workspace } from '../services/check-parser/package-files/workspace'
 
 export interface ProjectProps {
   /**
@@ -243,6 +244,7 @@ export class Session {
   static privateLocations: PrivateLocationApi[]
   static parsers = new Map<string, Parser>()
   static constructExports: ConstructExport[] = []
+  static workspace?: Workspace
 
   static async loadFile<T = unknown> (filePath: string): Promise<T> {
     const loader = this.loader
@@ -337,6 +339,7 @@ export class Session {
     const parser = new Parser({
       supportedNpmModules: Object.keys(runtime.dependencies),
       checkUnsupportedModules: Session.verifyRuntimeDependencies,
+      workspace: Session.workspace,
     })
 
     Session.parsers.set(runtime.name, parser)

packages/cli/src/services/__tests__/checkly-config-loader.spec.ts

@@ -2,7 +2,7 @@ import path from 'node:path'
 
 import { describe, it, expect } from 'vitest'
 
-import { loadChecklyConfig } from '../checkly-config-loader'
+import { loadChecklyConfig, defaultFilenames } from '../checkly-config-loader'
 import { splitConfigFilePath } from '../util'
 
 describe('loadChecklyConfig()', () => {
@@ -25,8 +25,10 @@ describe('loadChecklyConfig()', () => {
     try {
       await loadChecklyConfig(configDir)
     } catch (e: any) {
-      expect(e.message).toContain(`Unable to locate a config at ${configDir} with ${
-        ['checkly.config.ts', 'checkly.config.mts', 'checkly.config.cts', 'checkly.config.js', 'checkly.config.mjs', 'checkly.config.cjs'].join(', ')}.`)
+      expect(e.message).toContain(`Unable to detect a Checkly configuration file`)
+      for (const filename of defaultFilenames) {
+        expect(e.message).toContain(filename)
+      }
     }
   })
   it('config TS file should export an object', async () => {

packages/cli/src/services/check-parser/__tests__/check-parser-fixtures/workspace-example/package-lock.json

@@ -0,0 +1,7 @@
+{
+  "name": "workspace-example",
+  "version": "1.0.0",
+  "workspaces": [
+    "packages/*"
+  ]
+}

packages/cli/src/services/check-parser/__tests__/check-parser-fixtures/workspace-example/package.json

@@ -0,0 +1,14 @@
+import { defineConfig } from 'checkly'
+
+const config = defineConfig({
+  projectName: 'Workspace Example Project',
+  logicalId: '56fbaf4d-fc2c-418c-868a-3f461809ed37',
+  checks: {
+    checkMatch: '**/__checks__/**/*.check.ts',
+    tags: [
+      'mac',
+    ],
+  },
+})
+
+export default config