@ouraihub/aether-core 0.1.0 → 0.1.2

package/src/search/policy.ts

@@ -15,7 +15,10 @@ export const PROVIDERS: readonly ProviderSpec[] = [
     name: '智谱 Zhipu',
     description: 'Web search',
     basicAuth: false,
-    search: { defaultBaseUrl: 'https://open.bigmodel.cn/api/paas/v4/web_search', keyMode: 'required' },
+    search: {
+      defaultBaseUrl: 'https://open.bigmodel.cn/api/paas/v4/web_search',
+      keyMode: 'required',
+    },
   },
   {
     id: 'tavily',
@@ -98,7 +101,10 @@ export const PROVIDERS: readonly ProviderSpec[] = [
     fetch: { defaultBaseUrl: '', keyMode: 'none' },
   },
 ]
-export const DEFAULT_PROVIDERS: Readonly<Record<SearchCap, string>> = { search: 'exa-mcp', fetch: 'jina' }
+export const DEFAULT_PROVIDERS: Readonly<Record<SearchCap, string>> = {
+  search: 'exa-mcp',
+  fetch: 'jina',
+}
 export const defaultPreferences = (): SearchPreferences => ({
   maxResults: 5,
   compression: 'none',
@@ -141,7 +147,12 @@ export function targetUrl(value: string): URL {
   return url
 }
 export function normalizeRule(value: unknown): string {
-  if (typeof value !== 'string' || !value || [...value].length > 256 || /[\s[\]\\^{}|#]/u.test(value))
+  if (
+    typeof value !== 'string' ||
+    !value ||
+    [...value].length > 256 ||
+    /[\s[\]\\^{}|#]/u.test(value)
+  )
     throw badRequest('blacklist', 'invalid')
   for (const character of value) {
     const code = character.codePointAt(0)
@@ -150,25 +161,39 @@ export function normalizeRule(value: unknown): string {
   const parts = value.match(/^(https?):\/\/([^/]+)(\/.*)?$/i)
   const hostPort = parts ? (parts[2] ?? '') : value
   const match = hostPort.match(/^([^:]+)(?::([0-9]+))?$/)
-  if (!match || (!parts && match[2]) || (match[2] && (Number(match[2]) < 1 || Number(match[2]) > 65535)))
+  if (
+    !match ||
+    (!parts && match[2]) ||
+    (match[2] && (Number(match[2]) < 1 || Number(match[2]) > 65535))
+  )
     throw badRequest('blacklist', 'invalid')
   const host = (match[1] ?? '').toLowerCase().replace(/\.$/, '')
   if (
     host.length > 253 ||
     host
       .split('.')
-      .some((label) => !label || label.length > 63 || !/^[a-z0-9*?](?:[a-z0-9*?-]*[a-z0-9*?])?$/u.test(label))
+      .some(
+        (label) =>
+          !label || label.length > 63 || !/^[a-z0-9*?](?:[a-z0-9*?-]*[a-z0-9*?])?$/u.test(label),
+      )
   )
     throw badRequest('blacklist', 'invalid')
   const scheme = parts?.[1]?.toLowerCase()
   const port =
-    match[2] && !((scheme === 'https' && match[2] === '443') || (scheme === 'http' && match[2] === '80'))
+    match[2] &&
+    !((scheme === 'https' && match[2] === '443') || (scheme === 'http' && match[2] === '80'))
       ? ':' + match[2]
       : ''
   return parts ? scheme + '://' + host + port + (parts[3] ?? '/') : host
 }
 export function validatePreferences(value: unknown): SearchPreferences {
-  const input = fields(value, ['maxResults', 'compression', 'cutoffLimit', 'blacklist', 'modelToolsPreferred'])
+  const input = fields(value, [
+    'maxResults',
+    'compression',
+    'cutoffLimit',
+    'blacklist',
+    'modelToolsPreferred',
+  ])
   const { maxResults, compression, cutoffLimit, blacklist, modelToolsPreferred } = input
   if (
     typeof maxResults !== 'number' ||
@@ -238,7 +263,10 @@ export function truncate(text: string, limit: number): string {
 export function cutoff(text: string, preferences: SearchPreferences): string {
   return preferences.compression === 'cutoff' ? truncate(text, preferences.cutoffLimit) : text
 }
-export function processHits(hits: readonly SearchHit[], preferences: SearchPreferences): SearchHit[] {
+export function processHits(
+  hits: readonly SearchHit[],
+  preferences: SearchPreferences,
+): SearchHit[] {
   const seen = new Set<string>()
   const result: SearchHit[] = []
   for (const hit of hits) {
@@ -251,7 +279,11 @@ export function processHits(hits: readonly SearchHit[], preferences: SearchPrefe
     }
     if (seen.has(url)) continue
     seen.add(url)
-    result.push({ title: truncate(hit.title || url, 500), url, snippet: cutoff(hit.snippet, preferences) })
+    result.push({
+      title: truncate(hit.title || url, 500),
+      url,
+      snippet: cutoff(hit.snippet, preferences),
+    })
     if (result.length >= preferences.maxResults) break
   }
   return result
@@ -262,6 +294,7 @@ export function validateQuery(value: string): string {
   return query
 }
 export function validateTarget(value: string): string {
-  if (new TextEncoder().encode(stringValue(value, 'url')).length > 8192) throw badRequest('url', 'too_long', 8192)
+  if (new TextEncoder().encode(stringValue(value, 'url')).length > 8192)
+    throw badRequest('url', 'too_long', 8192)
   return targetUrl(value.trim()).href
 }

package/src/search/vendor.ts

@@ -53,7 +53,16 @@ function collect(items: readonly unknown[]): SearchHit[] {
     hits.push({
       title: text(item, 'title', 'name') || url,
       url,
-      snippet: text(item, 'content', 'text', 'snippet', 'description', 'excerpts', 'summary', 'markdown'),
+      snippet: text(
+        item,
+        'content',
+        'text',
+        'snippet',
+        'description',
+        'excerpts',
+        'summary',
+        'markdown',
+      ),
     })
   }
   return hits
@@ -116,7 +125,10 @@ export function parseMcp(body: string): SearchHit[] {
       })),
   )
 }
-function headers(key: string | undefined, kind: 'bearer' | 'api' = 'bearer'): Record<string, string> {
+function headers(
+  key: string | undefined,
+  kind: 'bearer' | 'api' = 'bearer',
+): Record<string, string> {
   return {
     accept: 'application/json',
     'content-type': 'application/json',
@@ -155,7 +167,9 @@ export function searchRequest(
       init = {
         headers: {
           accept: 'application/json',
-          ...(config.basicAuthUsername ? { authorization: basic(config.basicAuthUsername, key ?? '') } : {}),
+          ...(config.basicAuthUsername
+            ? { authorization: basic(config.basicAuthUsername, key ?? '') }
+            : {}),
         },
       }
       break
@@ -194,7 +208,10 @@ export function searchRequest(
       break
     case 'parallel':
       url += '/v1/search'
-      init = post({ objective: query, search_queries: [query], advanced_settings: { max_results: count } }, 'api')
+      init = post(
+        { objective: query, search_queries: [query], advanced_settings: { max_results: count } },
+        'api',
+      )
       break
     case 'jina':
       url += '/' + encodeURIComponent(query)
@@ -205,7 +222,10 @@ export function searchRequest(
       init = post({ query, limit: count, scrapeOptions: { formats: ['markdown'] } })
       break
     default:
-      throw new KitFailure('bad_request', 'Unsupported search provider', { field: 'provider', reason: 'unsupported' })
+      throw new KitFailure('bad_request', 'Unsupported search provider', {
+        field: 'provider',
+        reason: 'unsupported',
+      })
   }
   return { url, init }
 }
@@ -226,16 +246,26 @@ export function fetchRequest(
         },
       }
     case 'jina':
-      return { url: base + '/' + target, init: { headers: { ...headers(key), 'x-retain-images': 'none' } } }
+      return {
+        url: base + '/' + target,
+        init: { headers: { ...headers(key), 'x-retain-images': 'none' } },
+      }
     case 'firecrawl':
       return {
         url: base + '/v2/scrape',
-        init: { method: 'POST', headers: headers(key), body: JSON.stringify({ url: target, formats: ['markdown'] }) },
+        init: {
+          method: 'POST',
+          headers: headers(key),
+          body: JSON.stringify({ url: target, formats: ['markdown'] }),
+        },
       }
     case 'fetch':
       return { url: target, init: { method: 'GET', headers: { accept: 'text/html, text/plain' } } }
     default:
-      throw new KitFailure('bad_request', 'Unsupported fetch provider', { field: 'provider', reason: 'unsupported' })
+      throw new KitFailure('bad_request', 'Unsupported fetch provider', {
+        field: 'provider',
+        reason: 'unsupported',
+      })
   }
 }
 export function parsePage(provider: string, body: string, target: string): SearchPage {
@@ -246,7 +276,10 @@ export function parsePage(provider: string, body: string, target: string): Searc
         ?.replace(/<[^>]*>/gu, ' ')
         .replace(/\s+/gu, ' ')
         .trim() || target
-    const visible = body.replace(/<!--[\s\S]*?-->|<(script|style|noscript|head|title)\b[^>]*>[\s\S]*?<\/\1\s*>/giu, ' ')
+    const visible = body.replace(
+      /<!--[\s\S]*?-->|<(script|style|noscript|head|title)\b[^>]*>[\s\S]*?<\/\1\s*>/giu,
+      ' ',
+    )
     const documentBody = visible.match(/<body\b[^>]*>([\s\S]*?)(?:<\/body\s*>|$)/iu)?.[1] ?? visible
     const content = documentBody
       .replace(/<[^>]*>/gu, ' ')

package/src/skills/content.ts

@@ -14,15 +14,25 @@ export interface FileIdentity {
 /** docs14 §3.1: validated text and unique files in ascending path order are prerequisites.
  * No trimming, newline normalization, sorting or case conversion is performed here.
  */
-export async function digest(content: ContentIdentity, files: readonly FileIdentity[] = []): Promise<string> {
+export async function digest(
+  content: ContentIdentity,
+  files: readonly FileIdentity[] = [],
+): Promise<string> {
   const encoder = new TextEncoder()
-  const fields: (string | null)[] = [content.name, content.description, content.version, content.body]
+  const fields: (string | null)[] = [
+    content.name,
+    content.description,
+    content.version,
+    content.body,
+  ]
   if (content.license !== null || files.length !== 0) {
     fields.push('package', content.license)
     for (const file of files) fields.push(file.path, file.sha256)
   }
   const encoded = fields.map((field) => (field === null ? null : encoder.encode(field)))
-  const bytes = new Uint8Array(encoded.reduce((size, field) => size + (field === null ? 1 : 9 + field.length), 0))
+  const bytes = new Uint8Array(
+    encoded.reduce((size, field) => size + (field === null ? 1 : 9 + field.length), 0),
+  )
   const view = new DataView(bytes.buffer)
   let offset = 0
   for (const field of encoded) {

package/src/skills/index.ts

@@ -1,6 +1,11 @@
 export { digest, sha256 } from './content.js'
 export type { ContentIdentity, FileIdentity } from './content.js'
-export { parseManualSave, parseSkillMarkdown, parseSkillSource, validateSkillContent } from './validation.js'
+export {
+  parseManualSave,
+  parseSkillMarkdown,
+  parseSkillSource,
+  validateSkillContent,
+} from './validation.js'
 export { readSkillSource } from './package.js'
 export type { PackageFile, SkillPackage } from './package.js'
 export type { BuiltinSkill, SkillSnapshot, SkillRecord, SkillCatalog } from './service.js'

package/src/skills/package.ts

@@ -1,7 +1,12 @@
 import type { SkillFileKind } from '@ouraihub/aether-contracts'
 import { badRequest, KitFailure } from '../error.js'
 import { sha256 } from './content.js'
-import { checkSecrets, parseSkillMarkdown, parseSkillSource, type ParsedSkill } from './validation.js'
+import {
+  checkSecrets,
+  parseSkillMarkdown,
+  parseSkillSource,
+  type ParsedSkill,
+} from './validation.js'
 
 export interface PackageFile {
   readonly path: string
@@ -40,7 +45,13 @@ function normalize(path: string): string {
 }
 function kind(path: string): SkillFileKind | null {
   const top = path.split('/')[0]
-  return top === 'scripts' ? 'script' : top === 'references' ? 'reference' : top === 'assets' ? 'asset' : null
+  return top === 'scripts'
+    ? 'script'
+    : top === 'references'
+      ? 'reference'
+      : top === 'assets'
+        ? 'asset'
+        : null
 }
 function checkPath(path: string): void {
   if (!path.split('/').every((part) => /^[A-Za-z0-9_-][A-Za-z0-9._-]*$/u.test(part)))
@@ -82,7 +93,8 @@ function entries(bytes: Uint8Array): Entry[] {
   const result: Entry[] = []
   const paths = new Set<string>()
   for (let index = 0; index < count; index++) {
-    if (offset + 46 > centralEnd || view.getUint32(offset, true) !== 0x02014b50) throw badRequest('zip', 'invalid')
+    if (offset + 46 > centralEnd || view.getUint32(offset, true) !== 0x02014b50)
+      throw badRequest('zip', 'invalid')
     const flags = view.getUint16(offset + 8, true)
     if ((flags & 1) !== 0 || (flags & 64) !== 0) throw badRequest('zip', 'encrypted')
     const mode = view.getUint32(offset + 38, true) >>> 16
@@ -91,7 +103,8 @@ function entries(bytes: Uint8Array): Entry[] {
     const extraLength = view.getUint16(offset + 30, true)
     const commentLength = view.getUint16(offset + 32, true)
     const next = offset + 46 + nameLength + extraLength + commentLength
-    if (next > centralEnd || view.getUint16(offset + 34, true) !== 0) throw badRequest('zip', 'invalid')
+    if (next > centralEnd || view.getUint16(offset + 34, true) !== 0)
+      throw badRequest('zip', 'invalid')
     const rawPath = decode(bytes.subarray(offset + 46, offset + 46 + nameLength), 'zip')
     const path = normalize(rawPath)
     if (paths.has(path)) throw badRequest('zip', 'duplicate_path')
@@ -182,7 +195,10 @@ export async function readSkillSource(value: unknown): Promise<SkillPackage> {
     const fileEntries = allEntries.filter((entry) => !entry.directory)
     const first = fileEntries[0]?.path
     const prefix = first?.includes('/') ? first.slice(0, first.indexOf('/') + 1) : null
-    const strip = prefix !== null && fileEntries.every((entry) => entry.path.startsWith(prefix)) ? prefix.length : 0
+    const strip =
+      prefix !== null && fileEntries.every((entry) => entry.path.startsWith(prefix))
+        ? prefix.length
+        : 0
     const files: PackageFile[] = []
     const unsupported: string[] = []
     let raw: string | undefined
@@ -207,7 +223,8 @@ export async function readSkillSource(value: unknown): Promise<SkillPackage> {
       const limit = Math.min(ownLimit, TOTAL - total)
       const field = limit < ownLimit ? 'total' : path
       const content = await inflate(bytes, entry, limit, field)
-      if (content.length !== entry.size || crc32(content) !== entry.crc) throw badRequest('zip', 'invalid')
+      if (content.length !== entry.size || crc32(content) !== entry.crc)
+        throw badRequest('zip', 'invalid')
       total += content.length
       let value: string | undefined
       if (fileKind === 'asset') {
@@ -223,7 +240,12 @@ export async function readSkillSource(value: unknown): Promise<SkillPackage> {
         if (license !== null) throw badRequest(path, 'duplicate')
         license = value ?? null
       } else if (fileKind !== null)
-        files.push({ path, kind: fileKind, bytes: Array.from(content), sha256: await sha256(content) })
+        files.push({
+          path,
+          kind: fileKind,
+          bytes: Array.from(content),
+          sha256: await sha256(content),
+        })
     }
     if (unsupported.length) throw badRequest(unsupported.sort().join(', '), 'unsupported_files')
     if (raw === undefined) throw badRequest('SKILL.md', 'missing')

package/src/skills/service.ts

@@ -33,8 +33,12 @@ export interface SkillRecord extends SkillPackage {
 }
 export type SkillCatalog = readonly SkillRecord[]
 export const conflict = (reason: string, entity?: string): KitFailure =>
-  new KitFailure('conflict', 'Skill conflict', { reason, ...(entity === undefined ? {} : { entity }) })
-const missing = (entity: string): KitFailure => new KitFailure('not_found', 'Skill not found', { entity })
+  new KitFailure('conflict', 'Skill conflict', {
+    reason,
+    ...(entity === undefined ? {} : { entity }),
+  })
+const missing = (entity: string): KitFailure =>
+  new KitFailure('not_found', 'Skill not found', { entity })
 export function fileMetas(skill: SkillPackage): SkillFileMeta[] {
   return skill.files.map((file) => ({ path: file.path, kind: file.kind, size: file.bytes.length }))
 }
@@ -79,7 +83,12 @@ function pinned(catalog: SkillCatalog, id: string, expected: number): SkillRecor
 export function files(catalog: SkillCatalog, id: string, expected: number): SkillFileMeta[] {
   return fileMetas(pinned(catalog, id, expected))
 }
-export function file(catalog: SkillCatalog, id: string, expected: number, path: string): Uint8Array<ArrayBuffer> {
+export function file(
+  catalog: SkillCatalog,
+  id: string,
+  expected: number,
+  path: string,
+): Uint8Array<ArrayBuffer> {
   const found = pinned(catalog, id, expected).files.find((entry) => entry.path === path)
   if (!found) throw missing('skill_file')
   return new Uint8Array(found.bytes)
@@ -97,8 +106,20 @@ function ensureFree(catalog: SkillCatalog, name: string, except?: string): void
 function mutable(skill: SkillRecord): void {
   if (skill.origin === 'builtin') throw badRequest('id', 'builtin')
 }
-async function create(content: SkillPackage, id: string, origin: SkillOrigin, now: number): Promise<SkillRecord> {
-  return { ...content, id, origin, revision: 1, digest: await digest(content, content.files), updatedAt: now }
+async function create(
+  content: SkillPackage,
+  id: string,
+  origin: SkillOrigin,
+  now: number,
+): Promise<SkillRecord> {
+  return {
+    ...content,
+    id,
+    origin,
+    revision: 1,
+    digest: await digest(content, content.files),
+    updatedAt: now,
+  }
 }
 async function update(
   previous: SkillRecord,
@@ -157,12 +178,20 @@ export async function save(
   newId: string,
   now: number,
 ): Promise<SkillCatalog> {
-  const fields = { name: input.name, description: input.description, version: input.version ?? null, body: input.body }
+  const fields = {
+    name: input.name,
+    description: input.description,
+    version: input.version ?? null,
+    body: input.body,
+  }
   validateSkillContent(fields)
   if (input.id === undefined) {
     ensureFree(catalog, input.name)
     ensureId(catalog, newId)
-    return [...catalog, await create({ ...fields, license: null, extra: {}, files: [] }, newId, 'manual', now)]
+    return [
+      ...catalog,
+      await create({ ...fields, license: null, extra: {}, files: [] }, newId, 'manual', now),
+    ]
   }
   if (input.expectedRevision === undefined) throw badRequest('expectedRevision', 'missing')
   const current = record(catalog, input.id)
@@ -231,7 +260,10 @@ export async function syncBuiltins(
       result = [...result, await create(content, id, 'builtin', now)]
     }
   }
-  return result.length === catalog.length && result.every((skill, index) => skill === catalog[index]) ? catalog : result
+  return result.length === catalog.length &&
+    result.every((skill, index) => skill === catalog[index])
+    ? catalog
+    : result
 }
 
 /** Strictly narrows persisted documents before exposing their contents to services. */
@@ -260,16 +292,20 @@ export function parseCatalog(value: unknown): SkillCatalog {
       'skill',
     )
     const origin = entry.origin
-    if (origin !== 'builtin' && origin !== 'manual' && origin !== 'imported') throw badRequest('origin', 'invalid')
+    if (origin !== 'builtin' && origin !== 'manual' && origin !== 'imported')
+      throw badRequest('origin', 'invalid')
     const extras = object(entry.extra, 'extra')
-    const extra = Object.fromEntries(Object.entries(extras).map(([key, value]) => [key, text(value, 'extra')]))
+    const extra = Object.fromEntries(
+      Object.entries(extras).map(([key, value]) => [key, text(value, 'extra')]),
+    )
     const fileValues: unknown = entry.files
     if (!Array.isArray(fileValues)) throw badRequest('files', 'invalid')
     const parsedFiles: PackageFile[] = fileValues.map((rawFile: unknown) => {
       const file = object(rawFile, 'file')
       exact(file, ['path', 'kind', 'sha256', 'bytes'], 'file')
       const kind = file.kind
-      if (kind !== 'script' && kind !== 'reference' && kind !== 'asset') throw badRequest('kind', 'invalid')
+      if (kind !== 'script' && kind !== 'reference' && kind !== 'asset')
+        throw badRequest('kind', 'invalid')
       const bytes: unknown = file.bytes
       if (!Array.isArray(bytes)) throw badRequest('bytes', 'invalid')
       const content = bytes.map((byte: unknown) => {
@@ -281,10 +317,15 @@ export function parseCatalog(value: unknown): SkillCatalog {
       if (!/^[0-9a-f]{64}$/u.test(hash)) throw badRequest('sha256', 'invalid')
       return { path: text(file.path, 'path'), kind, sha256: hash, bytes: content }
     })
-    if (parsedFiles.some((file, index) => index > 0 && file.path <= (parsedFiles[index - 1]?.path ?? '')))
+    if (
+      parsedFiles.some(
+        (file, index) => index > 0 && file.path <= (parsedFiles[index - 1]?.path ?? ''),
+      )
+    )
       throw badRequest('files', 'invalid')
     const updatedAt = entry.updatedAt
-    if (typeof updatedAt !== 'number' || !Number.isSafeInteger(updatedAt)) throw badRequest('updatedAt', 'invalid')
+    if (typeof updatedAt !== 'number' || !Number.isSafeInteger(updatedAt))
+      throw badRequest('updatedAt', 'invalid')
     const skill: SkillRecord = {
       id: text(entry.id, 'id'),
       name: text(entry.name, 'name'),

package/src/skills/validation.ts

@@ -14,8 +14,13 @@ export function object(value: unknown, field: string): Record<string, unknown> {
   if (!isObject(value)) throw badRequest(field, 'invalid')
   return value
 }
-export function exact(value: Record<string, unknown>, keys: readonly string[], field: string): void {
-  if (Object.keys(value).some((key) => !keys.includes(key))) throw badRequest(field, 'unknown_field')
+export function exact(
+  value: Record<string, unknown>,
+  keys: readonly string[],
+  field: string,
+): void {
+  if (Object.keys(value).some((key) => !keys.includes(key)))
+    throw badRequest(field, 'unknown_field')
 }
 export function text(value: unknown, field: string): string {
   if (typeof value !== 'string') throw badRequest(field, 'invalid')
@@ -31,7 +36,17 @@ export function strings(value: unknown, field: string): string[] {
   return value.map((entry: unknown) => text(entry, field))
 }
 export function checkSecrets(value: string, field: string): void {
-  const prefixes = ['sk-', 'ghp_', 'gho_', 'github_pat_', 'xoxb-', 'xoxp-', 'AKIA', 'AIza', 'glpat-']
+  const prefixes = [
+    'sk-',
+    'ghp_',
+    'gho_',
+    'github_pat_',
+    'xoxb-',
+    'xoxp-',
+    'AKIA',
+    'AIza',
+    'glpat-',
+  ]
   let lineNumber = 0
   for (const line of value.split('\n')) {
     lineNumber++
@@ -40,17 +55,22 @@ export function checkSecrets(value: string, field: string): void {
     if (
       (line.includes('-----BEGIN') && line.includes('PRIVATE KEY')) ||
       (bearer?.[1] !== undefined && /^[A-Za-z0-9_./+=-]{20}/u.test(bearer[1])) ||
-      tokens.some((token) => token.length >= 20 && prefixes.some((prefix) => token.startsWith(prefix)))
+      tokens.some(
+        (token) => token.length >= 20 && prefixes.some((prefix) => token.startsWith(prefix)),
+      )
     ) {
       throw badRequest(`${field}:${lineNumber}`, 'secret_detected')
     }
   }
 }
-export function validateSkillContent(skill: Pick<ContentIdentity, 'name' | 'description' | 'version' | 'body'>): void {
+export function validateSkillContent(
+  skill: Pick<ContentIdentity, 'name' | 'description' | 'version' | 'body'>,
+): void {
   if (!/^[a-z0-9][a-z0-9-]{0,63}$/u.test(skill.name)) throw badRequest('name', 'invalid')
   if ([...skill.description].length < 1 || [...skill.description].length > 500)
     throw badRequest('description', 'invalid', 500)
-  if (skill.version !== null && [...skill.version].length > 32) throw badRequest('version', 'invalid', 32)
+  if (skill.version !== null && [...skill.version].length > 32)
+    throw badRequest('version', 'invalid', 32)
   if (!skill.body.trim()) throw badRequest('body', 'empty')
   if (encoder.encode(skill.body).length > 65536) throw badRequest('body', 'too_large', 65536)
   checkSecrets(skill.name, 'name')
@@ -65,7 +85,8 @@ export function parseManualSave(value: unknown): ManualSave {
     name: text(input.name, 'name'),
     description: text(input.description, 'description'),
     body: text(input.body, 'body'),
-    version: input.version === undefined || input.version === null ? null : text(input.version, 'version'),
+    version:
+      input.version === undefined || input.version === null ? null : text(input.version, 'version'),
     ...(input.id === undefined ? {} : { id: text(input.id, 'id') }),
     ...(input.expectedRevision === undefined
       ? {}
@@ -136,7 +157,8 @@ export function parseSkillMarkdown(raw: string): ParsedSkill {
     fields.set(key, value)
   }
   if (body === undefined) throw badRequest(String(lineNumber + 1), 'frontmatter_unsupported')
-  if (body.split('\n', 1)[0] === '---') throw badRequest(String(lineNumber + 1), 'frontmatter_unsupported')
+  if (body.split('\n', 1)[0] === '---')
+    throw badRequest(String(lineNumber + 1), 'frontmatter_unsupported')
   const name = fields.get('name')
   const description = fields.get('description')
   if (name === undefined) throw badRequest('name', 'missing')
@@ -145,7 +167,14 @@ export function parseSkillMarkdown(raw: string): ParsedSkill {
   fields.delete('name')
   fields.delete('description')
   fields.delete('version')
-  const skill: ParsedSkill = { name, description, version, body, license: null, extra: Object.fromEntries(fields) }
+  const skill: ParsedSkill = {
+    name,
+    description,
+    version,
+    body,
+    license: null,
+    extra: Object.fromEntries(fields),
+  }
   validateSkillContent(skill)
   return skill
 }