fix: use bare identifier references for build-time define constants in bootstrap.ts

Bun's `define` option performs compile-time identifier substitution — it
replaces bare identifier tokens in source code with their configured values.
Dynamic property lookups like `(globalThis as any)["ALTIMATE_VALIDATE_SKILL_MD"]`
are runtime expressions and are NOT substituted by the bundler.

The previous `readAsset(globalName: string, ...)` helper passed constant names
as strings, so the globalThis lookup always returned `undefined` in the
production binary. The fallback `fs.readFile(import.meta.dir + ...)` then
failed because the source paths don't exist in a compiled binary. Since
`ensureValidationSetup()` is wrapped in a bare `catch {}`, this failed
completely silently — skill files were never written to `~/.claude/` on startup.

Fix:
- Add `declare const` for all three build-time globals
- Change `readAsset` signature to accept the resolved value directly
- Pass bare identifiers at each call site so the bundler inlines the content

This matches the pattern already used correctly in `validate.ts`.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: Bharatram-altimate-ai

packages/opencode/src/project/bootstrap.ts

@@ -25,9 +25,14 @@ function getClaudeDir(): string {
   return path.join(os.homedir(), ".claude")
 }
 
-async function readAsset(globalName: string, fallbackRelPath: string): Promise<string> {
-  const val = (globalThis as any)[globalName]
-  if (typeof val === "string" && val) return val
+// Injected at build time by build.ts via Bun's define option.
+// Must be referenced as bare identifiers — dynamic globalThis lookup does not work with define.
+declare const ALTIMATE_VALIDATE_SKILL_MD: string
+declare const ALTIMATE_VALIDATE_BATCH_PY: string
+declare const ALTIMATE_LOGGER_HOOK_PY: string
+
+async function readAsset(defined: string, fallbackRelPath: string): Promise<string> {
+  if (typeof defined === "string" && defined) return defined
   return fs.readFile(path.join(import.meta.dir, fallbackRelPath), "utf-8")
 }
 
@@ -67,11 +72,11 @@ async function ensureValidationSetup(): Promise<void> {
     await fs.mkdir(validateSkillDir, { recursive: true })
     await fs.writeFile(
       path.join(validateSkillDir, "SKILL.md"),
-      await readAsset("ALTIMATE_VALIDATE_SKILL_MD", "../skill/validate/SKILL.md"),
+      await readAsset(ALTIMATE_VALIDATE_SKILL_MD, "../skill/validate/SKILL.md"),
     )
     await fs.writeFile(
       path.join(validateSkillDir, "batch_validate.py"),
-      await readAsset("ALTIMATE_VALIDATE_BATCH_PY", "../skill/validate/batch_validate.py"),
+      await readAsset(ALTIMATE_VALIDATE_BATCH_PY, "../skill/validate/batch_validate.py"),
     )
 
     // Install hook + register in settings.json only when logging is enabled
@@ -81,7 +86,7 @@ async function ensureValidationSetup(): Promise<void> {
       const hookPath = path.join(hooksDir, "altimate_logger_hook.py")
       await fs.writeFile(
         hookPath,
-        await readAsset("ALTIMATE_LOGGER_HOOK_PY", "../skill/validate/logger_hook.py"),
+        await readAsset(ALTIMATE_LOGGER_HOOK_PY, "../skill/validate/logger_hook.py"),
       )
       await mergeStopHook(
         path.join(claudeDir, "settings.json"),