@supatype/cli 0.1.7 → 0.1.10

package/tests/model-hooks.test.ts

@@ -33,7 +33,7 @@ function addFunction(dir: string, name: string): void {
   writeFileSync(join(dir, "functions", name, "index.ts"), "export default () => new Response()", "utf8")
 }
 
-describe("model hooks — declaration", () => {
+describe("model hooks: declaration", () => {
   it("extracts both the shorthand and the options form", () => {
     const { ast } = project(`
 import type { Model, UUID } from "@supatype/types"
@@ -84,7 +84,7 @@ export type Post = Model<{ id: UUID }, {
   })
 })
 
-describe("model hooks — push-time validation", () => {
+describe("model hooks: push-time validation", () => {
   const schema = `
 import type { Model, UUID } from "@supatype/types"
 
@@ -142,7 +142,7 @@ export type Post = Model<{ id: UUID }, { tableName: "posts" }>
   })
 })
 
-describe("model hooks — manifest map", () => {
+describe("model hooks: manifest map", () => {
   const schema = `
 import type { Model, UUID } from "@supatype/types"
 
@@ -193,7 +193,7 @@ export type Post = Model<{ id: UUID }, {
 
   it("never creates a manifest from scratch", () => {
     // `functions_enabled` is a plain bool on the server's side, so a manifest carrying only hooks
-    // would read as functions *disabled* — the defect fixed one commit earlier, by another door.
+    // would read as functions *disabled*, the defect fixed one commit earlier, by another door.
     const { dir, ast } = project(schema)
     expect(syncManifestHooks(dir, ast)).toBe(false)
     expect(existsSync(join(dir, ".supatype", "manifest.json"))).toBe(false)
@@ -220,7 +220,7 @@ export type Post = Model<{ id: UUID }, { tableName: "posts" }>
   })
 })
 
-describe("model hooks — generated module on disk", () => {
+describe("model hooks: generated module on disk", () => {
   it("removes a stale module when the last hook is deleted", () => {
     // Otherwise a typed module sits there claiming tables are hooked, and a handler importing it
     // keeps compiling while never running.
@@ -250,7 +250,7 @@ export type Post = Model<{ id: UUID }, { tableName: "posts"; hooks: { beforeChan
   })
 })
 
-describe("model hooks — doctor report", () => {
+describe("model hooks: doctor report", () => {
   const schema = `
 import type { Model, UUID } from "@supatype/types"
 

package/tests/runtime-contract.test.ts

@@ -130,7 +130,7 @@ describe("runtime contract", () => {
 
   // PostgREST must connect as `authenticator`, never as POSTGRES_USER. POSTGRES_USER is
   // `supatype_admin`, a superuser, and a superuser session may SET ROLE to any role in the
-  // cluster — so a request whose JWT named one got it. Verified against a live stack: a
+  // cluster: so a request whose JWT named one got it. Verified against a live stack: a
   // token with `role: "supatype_admin"` returned every row of an RLS-protected table.
   it("self-host compose connects PostgREST as authenticator, not the superuser", () => {
     const compose = renderSelfHostCompose(baseConfig)
@@ -149,7 +149,7 @@ describe("runtime contract", () => {
   })
 
   // `supatype dev --provider docker` renders *this* file, so a `${VAR:?}` with no default
-  // takes local dev down before anything starts — `docker compose` refuses to interpolate and
+  // takes local dev down before anything starts, `docker compose` refuses to interpolate and
   // exits 1. Adding one therefore means wiring it into `upsertDevComposeEnv` as well. This
   // list is the reminder; if it fails, do that rather than just updating the list.
   it("requires only variables the dev path also sets", () => {

package/tests/service-role-check.test.ts

@@ -66,7 +66,7 @@ describe("checkServiceRoleRoutes", () => {
   })
 
   it("warns rather than fails on a hook entry", () => {
-    // It was the documented form once, and the worker grants hooks whatever the list says — so it is
+    // It was the documented form once, and the worker grants hooks whatever the list says, so it is
     // harmless, and worth saying because a reader would assume the line is what grants.
     const problems = checkServiceRoleRoutes(config(["hooks/moderate-post"]), projectWith([]))
     expect(problems.errors).toEqual([])
@@ -89,7 +89,7 @@ describe("serviceRoleProblemLines", () => {
 
   it("lists what could have been named instead", () => {
     const lines = serviceRoleProblemLines({
-      errors: ['  "x" — no functions/x/index.ts'],
+      errors: ['  "x": no functions/x/index.ts'],
       missing: ["x"],
       warnings: [],
       available: ["audit", "send-email"],
@@ -99,7 +99,7 @@ describe("serviceRoleProblemLines", () => {
 
   it("says so when there are no functions at all", () => {
     const lines = serviceRoleProblemLines({
-      errors: ['  "x" — nope'],
+      errors: ['  "x": nope'],
       missing: ["x"],
       warnings: [],
       available: [],

package/tests/storage-provision.test.ts

@@ -80,7 +80,7 @@ describe("provisionBuckets", () => {
 
     await provisionBuckets("http://localhost:54321/storage/v1", "key", [{ id: "photos", public: false }])
 
-    expect(warn).toHaveBeenCalledWith('[storage] Storage API unreachable — skipped bucket "photos"')
+    expect(warn).toHaveBeenCalledWith('[storage] Storage API unreachable, skipped bucket "photos"')
   })
 })
 

package/tests/type-extractor.test.ts

@@ -1255,8 +1255,8 @@ export type Invoice = Model<{
 
   it("extracts Code and Currency as JSONB carrying their declared shape", () => {
     // Both were exported from `@supatype/types` and rejected by the extractor, so a schema using
-    // either failed the push with "unknown type". Each carries two values — a language with its
-    // source, an amount with its currency — so JSONB is the column that does not drop one.
+    // either failed the push with "unknown type". Each carries two values, a language with its
+    // source, an amount with its currency, so JSONB is the column that does not drop one.
     const dir = mkdtempSync(join(tmpdir(), "supatype-code-currency-"))
     dirs.push(dir)
     const schemaPath = join(dir, "schema.ts")