opencode-supabase 0.1.2-alpha.0 → 0.1.2-alpha.2

package/README.md

@@ -22,7 +22,7 @@ Connect your account and ask your agent about Supabase capabilities.
 
 ## Bundled Supabase Skills
 
-`opencode-supabase` ships the official Supabase agent skills by default:
+`opencode-supabase` ships the official [Supabase agent skills](https://github.com/supabase/agent-skills) by default:
 
 - `supabase`
 - `supabase-postgres-best-practices`
@@ -43,7 +43,7 @@ If you want Supabase tools without bundled skills, disable them in plugin option
 
 ### Select Individual Skills
 
-Per-skill config is a partial override. Omitted skills stay enabled.
+Set a bundled skill name to `false` to disable only that skill. Omitted skills stay enabled. Unknown keys are ignored with a warning.
 
 ```json
 {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-supabase",
-  "version": "0.1.2-alpha.0",
+  "version": "0.1.2-alpha.2",
   "type": "module",
   "description": "OpenCode plugin for Supabase integration with server and TUI components",
   "license": "Apache-2.0",

package/skills/supabase-postgres-best-practices/references/_contributing.md

@@ -27,7 +27,7 @@ to recognize anti-patterns.
 
 Include specific metrics. Helps agents prioritize fixes.
 
-**Good:** "10x faster queries", "50% smaller index", "Eliminates N+1" 
+**Good:** "10x faster queries", "50% smaller index", "Eliminates N+1"
 **Bad:** "Faster", "Better", "More efficient"
 
 ### 4. Self-Contained Examples

package/src/server/index.ts

@@ -13,7 +13,7 @@ const server: Plugin = async (input, options) => {
   return {
     config: async (config) => {
       registerSupabaseSkillPaths(config, options, {
-        warn: (message, data) => logger.warn(message, data as Record<string, unknown>),
+        warn: logger.warn,
       });
     },
     auth: createSupabaseAuth(input, options, { logger }),

package/src/server/skills.ts

@@ -1,5 +1,6 @@
 import fs from "node:fs";
 import path from "node:path";
+import { fileURLToPath } from "node:url";
 
 export const BUNDLED_SUPABASE_SKILLS = [
   "supabase",
@@ -8,7 +9,7 @@ export const BUNDLED_SUPABASE_SKILLS = [
 
 export type BundledSupabaseSkill = (typeof BUNDLED_SUPABASE_SKILLS)[number];
 
-type Warn = (message: string, data?: unknown) => void;
+type Warn = (message: string, data?: Record<string, unknown>) => unknown;
 
 type ResolverDeps = {
   warn?: Warn;
@@ -21,17 +22,21 @@ type RegisterDeps = ResolverDeps & {
 
 type ConfigWithSkills = object & {
   skills?: {
-    paths?: string[];
+    paths?: unknown;
   };
 };
 
+type SkillsConfig = {
+  paths?: unknown;
+} & Record<string, unknown>;
+
 function isRecord(value: unknown): value is Record<string, unknown> {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
 
 function pluginSkillsOption(options: unknown) {
   if (!isRecord(options) || !("skills" in options)) return true;
-  return options.skills;
+  return (options as { skills?: unknown }).skills;
 }
 
 export function resolveEnabledSupabaseSkills(options: unknown, deps: ResolverDeps = {}) {
@@ -48,6 +53,13 @@ export function resolveEnabledSupabaseSkills(options: unknown, deps: ResolverDep
   for (const key of Object.keys(value)) {
     if (!known.has(key)) {
       deps.warn?.("unknown Supabase bundled skill option ignored", { skill: key });
+      continue;
+    }
+    if (value[key] !== undefined && typeof value[key] !== "boolean") {
+      deps.warn?.("invalid Supabase bundled skill option value", {
+        skill: key,
+        value: value[key] as unknown,
+      });
     }
   }
 
@@ -55,7 +67,7 @@ export function resolveEnabledSupabaseSkills(options: unknown, deps: ResolverDep
 }
 
 export function defaultSkillsRoot() {
-  return path.resolve(path.dirname(new URL(import.meta.url).pathname), "../../skills");
+  return path.resolve(path.dirname(fileURLToPath(import.meta.url)), "../../skills");
 }
 
 export function registerSupabaseSkillPaths(
@@ -67,9 +79,31 @@ export function registerSupabaseSkillPaths(
   const skillsRoot = deps.skillsRoot ?? defaultSkillsRoot();
   const exists = deps.exists ?? fs.existsSync;
   const enabled = resolveEnabledSupabaseSkills(options, deps);
+  let skillsConfig: SkillsConfig;
+
+  if (configWithSkills.skills === undefined) {
+    skillsConfig = {};
+    configWithSkills.skills = skillsConfig;
+  } else if (isRecord(configWithSkills.skills)) {
+    skillsConfig = configWithSkills.skills as SkillsConfig;
+  } else {
+    deps.warn?.("invalid Supabase skills config; leaving unchanged", {
+      value: configWithSkills.skills as unknown,
+    });
+    return;
+  }
 
-  configWithSkills.skills = configWithSkills.skills ?? {};
-  configWithSkills.skills.paths = configWithSkills.skills.paths ?? [];
+  if (!Array.isArray(skillsConfig.paths)) {
+    if (skillsConfig.paths !== undefined) {
+      deps.warn?.("invalid Supabase skills.paths value; leaving unchanged", {
+        value: skillsConfig.paths as unknown,
+      });
+      return;
+    }
+    skillsConfig.paths = [];
+  }
+
+  const paths = skillsConfig.paths as string[];
 
   for (const skill of enabled) {
     const skillPath = path.join(skillsRoot, skill);
@@ -77,8 +111,9 @@ export function registerSupabaseSkillPaths(
       deps.warn?.("bundled Supabase skill directory not found", { skill, path: skillPath });
       continue;
     }
-    if (!configWithSkills.skills.paths.includes(skillPath)) {
-      configWithSkills.skills.paths.push(skillPath);
+
+    if (!paths.includes(skillPath)) {
+      paths.push(skillPath);
     }
   }
 }