@pikku/core 0.12.24 → 0.12.25

package/src/services/typed-variables-service.ts

@@ -20,20 +20,12 @@ export class TypedVariablesService<
     private variablesMeta: Record<string, VariableMeta>
   ) {}
 
-  get(
-    name: keyof TMap & string
-  ): Promise<string | undefined> | string | undefined
-  get(name: string): Promise<string | undefined> | string | undefined
-  get(name: string): Promise<string | undefined> | string | undefined {
-    return this.variables.get(name)
-  }
-
-  getJSON<K extends keyof TMap & string>(
+  get<K extends keyof TMap & string>(
     name: K
   ): Promise<TMap[K] | undefined> | TMap[K] | undefined
-  getJSON<T = unknown>(name: string): Promise<T | undefined> | T | undefined
-  getJSON(name: string): Promise<unknown> | unknown {
-    return this.variables.getJSON(name)
+  get<T = string>(name: string): Promise<T | undefined> | T | undefined
+  get(name: string): Promise<unknown> | unknown {
+    return this.variables.get(name)
   }
 
   getAll():
@@ -42,14 +34,10 @@ export class TypedVariablesService<
     return this.variables.getAll()
   }
 
-  set(name: string, value: string): Promise<void> | void {
+  set(name: string, value: unknown): Promise<void> | void {
     return this.variables.set(name, value)
   }
 
-  setJSON(name: string, value: unknown): Promise<void> | void {
-    return this.variables.setJSON(name, value)
-  }
-
   has(name: string): Promise<boolean> | boolean {
     return this.variables.has(name)
   }

package/src/services/variables-service.ts

@@ -1,11 +1,9 @@
 export interface VariablesService {
-  get: (name: string) => Promise<string | undefined> | string | undefined
-  getJSON: <T = unknown>(name: string) => Promise<T | undefined> | T | undefined
+  get<T = string>(name: string): Promise<T | undefined> | T | undefined
   getAll: () =>
     | Promise<Record<string, string | undefined>>
     | Record<string, string | undefined>
-  set: (name: string, value: string) => Promise<void> | void
-  setJSON: (name: string, value: unknown) => Promise<void> | void
+  set: (name: string, value: unknown) => Promise<void> | void
   has: (name: string) => Promise<boolean> | boolean
   delete: (name: string) => Promise<void> | void
 }

package/src/testing/service-tests.ts

@@ -782,13 +782,13 @@ export function defineServiceTests(config: ServiceTestConfig): void {
     const kek = 'test-key-encryption-key-32chars!'
 
     describe(`SecretService [${name}]`, () => {
-      test('setSecretJSON and getSecretJSON', async () => {
+      test('setSecret and getSecret', async () => {
         const service = await factory({ key: kek })
-        await service.setSecretJSON('api-key', {
+        await service.setSecret('api-key', {
           token: 'sk-123',
           endpoint: 'https://api.example.com',
         })
-        const result = await service.getSecretJSON<{
+        const result = await service.getSecret<{
           token: string
           endpoint: string
         }>('api-key')
@@ -800,9 +800,9 @@ export function defineServiceTests(config: ServiceTestConfig): void {
 
       test('getSecret returns raw string', async () => {
         const service = await factory({ key: kek })
-        await service.setSecretJSON('string-secret', 'plain-value')
+        await service.setSecret('string-secret', 'plain-value')
         const result = await service.getSecret('string-secret')
-        assert.strictEqual(result, '"plain-value"')
+        assert.strictEqual(result, 'plain-value')
       })
 
       test('hasSecret returns true/false', async () => {
@@ -818,17 +818,17 @@ export function defineServiceTests(config: ServiceTestConfig): void {
         })
       })
 
-      test('setSecretJSON upserts existing key', async () => {
+      test('setSecret upserts existing key', async () => {
         const service = await factory({ key: kek })
-        await service.setSecretJSON('upsert-key', { v: 1 })
-        await service.setSecretJSON('upsert-key', { v: 2 })
-        const result = await service.getSecretJSON<{ v: number }>('upsert-key')
+        await service.setSecret('upsert-key', { v: 1 })
+        await service.setSecret('upsert-key', { v: 2 })
+        const result = await service.getSecret<{ v: number }>('upsert-key')
         assert.deepEqual(result, { v: 2 })
       })
 
       test('deleteSecret removes the key', async () => {
         const service = await factory({ key: kek })
-        await service.setSecretJSON('to-delete', 'bye')
+        await service.setSecret('to-delete', 'bye')
         assert.strictEqual(await service.hasSecret('to-delete'), true)
         await service.deleteSecret('to-delete')
         assert.strictEqual(await service.hasSecret('to-delete'), false)
@@ -837,7 +837,7 @@ export function defineServiceTests(config: ServiceTestConfig): void {
       test('rotateKEK re-wraps all secrets', async () => {
         const newKEK = 'new-key-encryption-key-rotated!'
         const oldService = await factory({ key: kek })
-        await oldService.setSecretJSON('rotate-test', { important: 'data' })
+        await oldService.setSecret('rotate-test', { important: 'data' })
 
         const rotatedService = await factory({
           key: newKEK,
@@ -845,7 +845,7 @@ export function defineServiceTests(config: ServiceTestConfig): void {
           previousKey: kek,
         })
 
-        const before = await rotatedService.getSecretJSON<{
+        const before = await rotatedService.getSecret<{
           important: string
         }>('rotate-test')
         assert.deepEqual(before, { important: 'data' })
@@ -858,7 +858,7 @@ export function defineServiceTests(config: ServiceTestConfig): void {
           key: newKEK,
           keyVersion: 2,
         })
-        const after = await newOnlyService.getSecretJSON<{
+        const after = await newOnlyService.getSecret<{
           important: string
         }>('rotate-test')
         assert.deepEqual(after, { important: 'data' })

package/src/wirings/ai-agent/ai-agent-prepare.ts

@@ -624,7 +624,9 @@ export async function prepareAgentRun(
   }
 
   if (params.getCredential && agentRunner.withApiKey) {
-    const aiCredential = await params.getCredential<{ apiKey: string }>('AI_API_KEY')
+    const aiCredential = await params.getCredential<{ apiKey: string }>(
+      'AI_API_KEY'
+    )
     if (aiCredential?.apiKey?.trim()) {
       agentRunner = agentRunner.withApiKey(aiCredential.apiKey)
     }
@@ -707,7 +709,10 @@ export async function prepareAgentRun(
     agent.agentMode
   )
 
-  const instructions = await buildInstructions(resolvedName, packageName)
+  let instructions = await buildInstructions(resolvedName, packageName)
+  if (input.context) {
+    instructions = `${instructions}\n\nCurrent context (use these identifiers directly in tool calls — do not ask the user for them):\n${input.context}`
+  }
 
   const resolved = resolveModelConfig(resolvedName, agent)
 

package/src/wirings/ai-agent/ai-agent.types.ts

@@ -81,6 +81,10 @@ export interface AIAgentInput {
   attachments?: AIAgentInputAttachment[]
   model?: string
   temperature?: number
+  /** Structured context injected into the system instructions for this request.
+   *  Use to provide upfront state (e.g. current org/project/branch/deployment)
+   *  so the agent can call tools without asking the user for identifiers. */
+  context?: string
 }
 
 export interface AIAgentOutput {

package/src/wirings/oauth2/oauth2-client.test.ts

@@ -14,9 +14,8 @@ function createMockSecretService(
 ): SecretService & { getStoredSecrets: () => Map<string, unknown> } {
   const secrets = new Map<string, unknown>(Object.entries(initialSecrets))
   return {
-    getSecret: async (key: string) => secrets.get(key) as string,
-    getSecretJSON: async <T>(key: string) => secrets.get(key) as T,
-    setSecretJSON: async (key: string, value: unknown) => {
+    getSecret: async <T = string>(key: string) => secrets.get(key) as T,
+    setSecret: async (key: string, value: unknown) => {
       secrets.set(key, value)
     },
     deleteSecret: async (_key: string) => {

package/src/wirings/oauth2/oauth2-client.ts

@@ -124,7 +124,7 @@ export class OAuth2Client {
     }
 
     // Load from secrets
-    const token = await this.secrets.getSecretJSON<OAuth2Token>(
+    const token = await this.secrets.getSecret<OAuth2Token>(
       this.oauth2Config.tokenSecretId
     )
     this.cachedToken = token
@@ -209,7 +209,7 @@ export class OAuth2Client {
       scope: data.scope,
     }
 
-    await this.secrets.setSecretJSON(this.oauth2Config.tokenSecretId, token)
+    await this.secrets.setSecret(this.oauth2Config.tokenSecretId, token)
     this.cachedToken = token
 
     return token
@@ -233,7 +233,7 @@ export class OAuth2Client {
       return this.cachedAppCredential
     }
     this.cachedAppCredential =
-      await this.secrets.getSecretJSON<OAuth2AppCredential>(
+      await this.secrets.getSecret<OAuth2AppCredential>(
         this.appCredentialSecretId
       )
     return this.cachedAppCredential