@tscircuit/cli 0.0.144 → 0.0.161

package/{lib → cli/lib}/posthog.ts

@@ -1,14 +1,13 @@
-import { PostHog } from 'posthog-node'
+import { PostHog } from "posthog-node"
 
 const POSTHOG_API_KEY: string | undefined = process.env.POSTHOG_API_KEY
 
 let posthogInstance: PostHog | null = null
 
 if (POSTHOG_API_KEY) {
-  posthogInstance = new PostHog(
-    POSTHOG_API_KEY,
-    { host: 'https://us.i.posthog.com' }
-  )
+  posthogInstance = new PostHog(POSTHOG_API_KEY, {
+    host: "https://us.i.posthog.com",
+  })
 }
 
 const posthogProxy = new Proxy<PostHog>({} as PostHog, {
@@ -18,7 +17,7 @@ const posthogProxy = new Proxy<PostHog>({} as PostHog, {
     }
     // Return a no-op function for any method call if PostHog is not initialized
     return () => {}
-  }
+  },
 })
 
-export default posthogProxy
+export default posthogProxy

package/{lib → cli/lib}/soupify.ts

@@ -18,7 +18,7 @@ export const soupify = async (
     filePath: string
     exportName?: string
   },
-  ctx: Pick<AppContext, "runtime" | "params">
+  ctx: Pick<AppContext, "runtime" | "params">,
 ) => {
   debug(`reading ${filePath}`)
   const targetFileContent = await readFile(filePath, "utf-8")
@@ -38,13 +38,13 @@ export const soupify = async (
 
   if (!exportName) {
     throw new Error(
-      `Couldn't derive an export name and didn't find default export in "${filePath}"`
+      `Couldn't derive an export name and didn't find default export in "${filePath}"`,
     )
   }
 
   const tmpFilePath = Path.join(
     Path.dirname(filePath),
-    Path.basename(filePath).replace(/\.[^\.]+$/, "") + ".__tmp_entrypoint.tsx"
+    Path.basename(filePath).replace(/\.[^\.]+$/, "") + ".__tmp_entrypoint.tsx",
   )
 
   debug(`writing to ${tmpFilePath}`)
@@ -71,7 +71,7 @@ const elements = await createRoot().render(<Component />, projectBuilder)
 
 console.log(JSON.stringify(elements))
 
-`.trim()
+`.trim(),
   )
 
   debug(`using runtime ${ctx.runtime}`)

package/{lib → cli/lib}/util/app-context.ts

@@ -1,6 +1,6 @@
 import { AxiosInstance } from "axios"
 import Configstore from "configstore"
-import { ContextConfigProps } from "lib/create-config-manager"
+import { ContextConfigProps } from "cli/lib/create-config-manager"
 
 export type AppContext = {
   args: any

package/{lib → cli/lib}/util/create-context-and-run-program.ts

@@ -1,5 +1,5 @@
 import Configstore from "configstore"
-import { AppContext } from "./app-context"
+import type { AppContext } from "./app-context"
 import minimist from "minimist"
 import _ from "lodash"
 import prompts from "prompts"
@@ -7,10 +7,10 @@ import { perfectCli } from "perfect-cli"
 import { getProgram } from "../get-program"
 import defaultAxios from "axios"
 import kleur from "kleur"
-import { PARAM_HANDLERS_BY_PARAM_NAME } from "lib/param-handlers"
-import { createConfigHandler } from "lib/create-config-manager"
+import { PARAM_HANDLERS_BY_PARAM_NAME } from "cli/lib/param-handlers"
+import { createConfigHandler } from "cli/lib/create-config-manager"
 import dargs from "dargs"
-import { versionCmd } from "lib/cmd-fns/version"
+import { versionCmd } from "cli/lib/cmd-fns/version"
 
 export type CliArgs = {
   cmd: string[]
@@ -62,8 +62,8 @@ export const createContextAndRunProgram = async (process_args: any) => {
         kleur.grey(
           `[RES] ${res.status} ${res.config.method?.toUpperCase()} ${
             res.config.url
-          }`
-        )
+          }`,
+        ),
       )
       return res
     })
@@ -82,7 +82,11 @@ export const createContextAndRunProgram = async (process_args: any) => {
       // end ignores ---
 
       if (err.response?.status === 401) {
-        console.log(kleur.red("Authentication failed. Please run 'tsci login' to authenticate yourself."))
+        console.log(
+          kleur.red(
+            "Authentication failed. Please run 'tsci login' to authenticate yourself.",
+          ),
+        )
         process.exit(1)
       }
 
@@ -92,12 +96,12 @@ export const createContextAndRunProgram = async (process_args: any) => {
             err.config.url
           }\n\n${JSON.stringify(err.response?.data, null, "  ")}`
             .replace(/\\n/g, "\n")
-            .replace(/\\"/g, '"')
-        )
+            .replace(/\\"/g, '"'),
+        ),
       )
       console.log(kleur.yellow("[Request Body]:"), err.config.data)
       return Promise.reject(err)
-    }
+    },
   )
 
   const ctx: AppContext = {

package/{lib → cli/lib}/util/get-all-package-files.ts

@@ -13,7 +13,7 @@ import ignore from "ignore"
  * Returns an array of files paths.
  */
 export const getAllPackageFiles = async (
-  ctx: AppContext
+  ctx: AppContext,
 ): Promise<Array<string>> => {
   await ensureNodeModulesIgnored()
 
@@ -56,7 +56,7 @@ const ensureNodeModulesIgnored = async () => {
     if (!gitignore.includes("node_modules/")) {
       await fs.appendFile(gitignorePath, "\nnode_modules/\n")
     }
-  } catch (err) {
+  } catch (err: any) {
     if (err.code === "ENOENT") {
       await fs.writeFile(gitignorePath, "node_modules/\n")
     } else {

package/cli/lib/util/lint-project.ts

@@ -0,0 +1,137 @@
+import * as ts from "typescript"
+import * as fs from "fs"
+import * as path from "path"
+
+interface LintResult {
+  filePath: string
+  messages: LintMessage[]
+}
+
+interface LintMessage {
+  line: number
+  column: number
+  message: string
+  fix?: {
+    range: [number, number]
+    text: string
+  }
+}
+
+export function lintProject(
+  projectPath: string,
+  fix: boolean = false,
+): LintResult[] {
+  const results: LintResult[] = []
+  const files = getTypeScriptFiles(projectPath)
+
+  for (const file of files) {
+    const sourceFile = ts.createSourceFile(
+      file,
+      fs.readFileSync(file, "utf8"),
+      ts.ScriptTarget.Latest,
+      true,
+    )
+
+    const result = lintFile(sourceFile, fix)
+    if (result.messages.length > 0) {
+      results.push({ filePath: file, messages: result.messages })
+    }
+
+    if (fix && result.messages.some((m) => m.fix)) {
+      const newContent = applyFixes(sourceFile.text, result.messages)
+      fs.writeFileSync(file, newContent)
+    }
+  }
+
+  return results
+}
+
+function lintFile(
+  sourceFile: ts.SourceFile,
+  fix: boolean,
+): { messages: LintMessage[] } {
+  const messages: LintMessage[] = []
+
+  function visit(node: ts.Node) {
+    if (
+      ts.isCallExpression(node) &&
+      ts.isIdentifier(node.expression) &&
+      node.expression.text === "capacitor"
+    ) {
+      const valueArg = node.arguments.find((arg) =>
+        ts.isObjectLiteralExpression(arg),
+      )
+      if (valueArg && ts.isObjectLiteralExpression(valueArg)) {
+        const valueProp = valueArg.properties.find(
+          (prop) =>
+            ts.isPropertyAssignment(prop) &&
+            ts.isIdentifier(prop.name) &&
+            prop.name.text === "value",
+        )
+        if (
+          valueProp &&
+          ts.isPropertyAssignment(valueProp) &&
+          ts.isStringLiteral(valueProp.initializer)
+        ) {
+          const value = valueProp.initializer.text
+          if (!value.match(/[µuμ]F$/)) {
+            const { line, character } =
+              sourceFile.getLineAndCharacterOfPosition(valueProp.getStart())
+            messages.push({
+              line: line + 1,
+              column: character + 1,
+              message: 'Capacitor value should include units (e.g., "100F")',
+              fix: fix
+                ? {
+                    range: [
+                      valueProp.initializer.getStart(),
+                      valueProp.initializer.getEnd(),
+                    ],
+                    text: `"${value}F"`,
+                  }
+                : undefined,
+            })
+          }
+        }
+      }
+    }
+
+    ts.forEachChild(node, visit)
+  }
+
+  visit(sourceFile)
+
+  return { messages }
+}
+
+function getTypeScriptFiles(dir: string): string[] {
+  const files: string[] = []
+  const entries = fs.readdirSync(dir, { withFileTypes: true })
+
+  for (const entry of entries) {
+    const fullPath = path.join(dir, entry.name)
+    if (entry.isDirectory()) {
+      if (entry.name !== "node_modules" && entry.name !== "dist") {
+        files.push(...getTypeScriptFiles(fullPath))
+      }
+    } else if (entry.isFile() && /\.tsx?$/.test(entry.name)) {
+      files.push(fullPath)
+    }
+  }
+
+  return files
+}
+
+function applyFixes(source: string, messages: LintMessage[]): string {
+  const fixes = messages
+    .filter((m) => m.fix)
+    .sort((a, b) => b.fix!.range[0] - a.fix!.range[0])
+
+  let result = source
+  for (const message of fixes) {
+    const [start, end] = message.fix!.range
+    result = result.slice(0, start) + message.fix!.text + result.slice(end)
+  }
+
+  return result
+}

package/{tests → cli/tests}/init.test.ts

@@ -4,6 +4,6 @@ import { $ } from "bun"
 test.skip("tsci init", async () => {
   await $`rm -rf ./tests/example-init`
   console.log(
-    await $`bun cli.ts init --name init-test --dir ./tests/example-init`.text()
+    await $`bun cli.ts init --name init-test --dir ./tests/example-init`.text(),
   )
 })

package/{tests → cli/tests}/open.test.ts

@@ -1,7 +1,8 @@
 import { test, expect } from "bun:test"
 import { $ } from "bun"
 
-test("tsci open", async () => {
+// This test annoyingly opens a browser window, unskip if you're testing it
+test.skip("tsci open", async () => {
   const result = await $`bun cli.ts open -y --cwd ./example-project`.text()
   expect(result).toContain("http")
   expect(result).toContain("example-project")

package/{tests → cli/tests}/soupify.test.ts

@@ -3,7 +3,7 @@ import { $ } from "bun"
 
 test("soupify", async () => {
   const result =
-    await $`bun cli.ts soupify -y --file ./example-project/examples/basic-bug.tsx`.text()
+    await $`bun cli/cli.ts soupify -y --file ./example-project/examples/basic-chip.tsx`.text()
 
   expect(result).toContain("10kohm")
 })