kiosapi 0.1.13 → 0.1.17

package/dist/agent/run.js

@@ -1,5 +1,5 @@
 import { existsSync, readFileSync, unlinkSync, writeFileSync } from 'node:fs';
-import { extname } from 'node:path';
+import { extname, join } from 'node:path';
 import { chatComplete, generateImage, resolveMediaModel, streamVision, } from '../api.js';
 import { CHECKPOINT_PATH } from '../config.js';
 import { cyan, dim, green, idn, prompt, red, rupiah, thinking, yellow } from '../ui.js';
@@ -235,9 +235,12 @@ async function runTool(call, otomatis, model) {
             case 'baca_file':
                 console.log(dim(`  baca ${str('path')}`));
                 return { output: bacaFile(str('path')) };
-            case 'daftar_file':
-                console.log(dim(`  daftar ${str('path') || '.'}`));
-                return { output: daftarFile(str('path') || '.') };
+            case 'daftar_file': {
+                const ked = typeof args['kedalaman'] === 'number' ? args['kedalaman'] : undefined;
+                const dPath = str('path') || '.';
+                console.log(dim(`  daftar ${dPath}${ked != null ? ` (kedalaman ${ked})` : ''}`));
+                return { output: daftarFile(dPath, ked) };
+            }
             case 'cari':
                 console.log(dim(`  cari "${str('pola')}"`));
                 return { output: cari(str('pola'), str('ext') || undefined) };
@@ -370,6 +373,34 @@ export function undoLastTurn(s) {
  * agent's final text (last answer or the `selesai` summary) — useful for chaining agents in a team.
  */
 export async function runTurn(s, userText) {
+    // On the very first turn of a fresh session, append project metadata to the existing system
+    // message so the model starts oriented without needing daftar_file(".") for basic orientation.
+    // We APPEND (not push a new system message) because some providers reject requests that have
+    // more than one system-role message — a second {role:'system'} at index 1 causes HTTP 400.
+    if (s.messages.filter((m) => m.role === 'user').length === 0) {
+        const snippets = [];
+        try {
+            snippets.push(`<root-directory>\n${daftarFile('.')}\n</root-directory>`);
+        }
+        catch { /* ignore */ }
+        for (const f of ['package.json', 'pyproject.toml', 'Cargo.toml', 'go.mod', 'README.md']) {
+            const abs = join(process.cwd(), f);
+            if (existsSync(abs)) {
+                try {
+                    const content = readFileSync(abs, 'utf8');
+                    snippets.push(`<file path="${f}">\n${content.slice(0, 4000)}\n</file>`);
+                }
+                catch { /* unreadable — skip */ }
+            }
+        }
+        if (snippets.length > 0) {
+            const sysMsg = s.messages[0];
+            if (sysMsg?.role === 'system') {
+                sysMsg.content +=
+                    `\n\n## Konteks Proyek (auto-injected — jangan panggil daftar_file(".") lagi)\n${snippets.join('\n\n')}`;
+            }
+        }
+    }
     s.messages.push({ role: 'user', content: userText });
     const tools = toolsForMode(s.mode);
     let lastText = '';
@@ -387,6 +418,11 @@ export async function runTurn(s, userText) {
     const lastSigs = []; // last 3 sigs for consecutive detection
     const COUNT_LIMIT = 4; // same tool+args called 4× total → loop
     const CONSEC_LIMIT = 3; // same sig 3× in a row → loop
+    // Read-only tool cache: on 2nd+ identical call, return the previous result with a warning
+    // instead of re-running. This gives the model early feedback so it can self-correct before
+    // COUNT_LIMIT is reached, preventing the common "list root, list root, list root" pattern.
+    const READ_ONLY_TOOLS = new Set(['baca_file', 'daftar_file', 'cari', 'lihat_gambar']);
+    const toolCache = new Map(); // sig → first output
     const stepLimit = s.maxSteps ?? MAX_STEPS;
     for (let step = 0; step < stepLimit; step++) {
         const stop = thinking();
@@ -418,7 +454,13 @@ export async function runTurn(s, userText) {
         // (null content + no tool_calls) can appear from a truncated stream or a reasoning-only
         // step; pushing it corrupts the history and causes providers to reject subsequent calls.
         if (reply.content !== null || calls.length > 0) {
-            s.messages.push({ role: 'assistant', content: reply.content, tool_calls: reply.tool_calls });
+            // When tool_calls are present, set content to null regardless of what text the model
+            // prefaced the call with. The text was already streamed to the user via onText; keeping
+            // it in history causes strict OpenAI-compat providers (DeepSeek, Workers AI, etc.) to
+            // return HTTP 400 on the next call because they require content: null when tool_calls
+            // is non-empty. Anthropic also works correctly with null here.
+            const storedContent = calls.length > 0 ? null : reply.content;
+            s.messages.push({ role: 'assistant', content: storedContent, tool_calls: reply.tool_calls });
         }
         if (reply.content)
             lastText = reply.content;
@@ -466,10 +508,30 @@ export async function runTurn(s, userText) {
         }
         const stepModified = new Set();
         for (const call of calls) {
+            const sig = `${call.function.name}:${call.function.arguments}`;
+            const count = callCounts.get(sig) ?? 1;
+            // For read-only tools called more than once with the same args: return the cached result
+            // with an escalating warning instead of re-running. The model gets immediate feedback so
+            // it can self-correct (pick a subfolder, go deeper) rather than looping to COUNT_LIMIT.
+            if (count > 1 && READ_ONLY_TOOLS.has(call.function.name) && toolCache.has(sig)) {
+                const toolName = call.function.name;
+                const cachedOut = toolCache.get(sig) ?? '';
+                const warn = count >= COUNT_LIMIT - 1
+                    ? `⚠ STOP: "${toolName}" sudah dipanggil ${count}× dengan argumen yang sama — ` +
+                        `hasilnya tidak berubah. WAJIB gunakan tool "selesai" atau eksplorasi path BERBEDA.`
+                    : `[Cache — identik dengan panggilan sebelumnya]\n${cachedOut}\n\n` +
+                        `⚠ Kamu sudah memanggil "${toolName}" dengan argumen yang sama ${count}×. ` +
+                        `Jangan ulangi — masuk ke subfolder spesifik atau gunakan "selesai".`;
+                console.log(dim(`  ↩ ${toolName} (cache ke-${count})`));
+                s.messages.push({ role: 'tool', content: warn, tool_call_id: call.id });
+                continue;
+            }
             const result = await runTool(call, s.otomatis, s.model);
             s.messages.push({ role: 'tool', content: result.output, tool_call_id: call.id });
             if (result.modifiedPath)
                 stepModified.add(result.modifiedPath);
+            if (READ_ONLY_TOOLS.has(call.function.name))
+                toolCache.set(sig, result.output);
             if (result.done) {
                 if (stepModified.size > 0)
                     console.log(dim(`  ✎ ${[...stepModified].join(' · ')}`));

package/dist/agent/schemas.js

@@ -16,10 +16,16 @@ const TOOLS = {
         type: 'function',
         function: {
             name: 'daftar_file',
-            description: 'Daftar isi sebuah folder.',
+            description: 'Daftar isi sebuah folder sebagai tree. Default: 3 level untuk ".", 1 level untuk subfolder. Gunakan kedalaman=2-4 untuk subfolder yang dalam.',
             parameters: {
                 type: 'object',
-                properties: { path: { type: 'string', description: 'Path folder (default ".")' } },
+                properties: {
+                    path: { type: 'string', description: 'Path folder (default ".")' },
+                    kedalaman: {
+                        type: 'number',
+                        description: 'Kedalaman tree 1–5. Default: 3 untuk ".", 1 untuk subfolder.',
+                    },
+                },
             },
         },
     },
@@ -192,10 +198,11 @@ Direktori kerja: ${process.cwd()}
 OS: ${osName} · ${shellNote}
 Aturan:
 - Bekerja langkah demi langkah: pakai tool untuk membaca sebelum mengubah.
+- Strategi eksplorasi: daftar_file(".") sudah tersedia di konteks awal — identifikasi file yang relevan LANGSUNG, lalu baca dengan baca_file. Jangan ulangi daftar_file pada path yang sama.
+- JANGAN memanggil tool APAPUN dengan argumen identik lebih dari 1× dalam satu sesi. Jika tool mengembalikan peringatan cache "⚠", langsung ganti ke path atau argumen BERBEDA.
 - Path selalu relatif ke direktori kerja; akses ke luar ditolak.
 - Gunakan hapus_file/pindah_file untuk menghapus/memindahkan file (lebih aman dari jalankan del/rm).
 - Buat perubahan kecil dan jelas. Setelah tugas beres, panggil tool "selesai" dengan ringkasan.
-- JANGAN memanggil tool yang sama dengan argumen yang sama lebih dari 1× berturut-turut. Jika sudah mendapat hasil daftar_file atau baca_file, langsung lanjutkan — JANGAN ulangi panggilan yang sama.
-- Jika tidak tahu harus berbuat apa selanjutnya, gunakan tool "selesai" dan jelaskan apa yang sudah ditemukan.
+- Jika tidak tahu harus berbuat apa: gunakan tool "selesai" dan jelaskan apa yang sudah ditemukan.
 - Jawab dan jelaskan dalam Bahasa Indonesia.`;
 }

package/dist/agent/tools.js

@@ -52,20 +52,43 @@ export function bacaFile(path) {
         ? `${text.slice(0, MAX_READ)}\n…[dipotong, ${text.length} char total]`
         : text;
 }
-/** daftar_file — list a directory (ignored entries hidden). */
-export function daftarFile(path) {
+/** daftar_file — list a directory (ignored entries hidden).
+ *  kedalaman controls tree depth (1–5). Defaults: 3 for root ".", 1 for subdirs.
+ *  Output is capped at 300 lines to keep API payloads reasonable. */
+export function daftarFile(path, kedalaman) {
     const { abs, rel } = safePath(path || '.');
     if (!existsSync(abs))
         return `Error: folder tidak ada: ${path}`;
     const patterns = ignorePatterns();
-    const entries = readdirSync(abs)
-        .filter((name) => !isProtected(rel === '.' ? name : `${rel}/${name}`, patterns))
-        .sort()
-        .map((name) => {
-        const dir = statSync(join(abs, name)).isDirectory();
-        return dir ? `${name}/` : name;
-    });
-    return entries.length > 0 ? entries.join('\n') : '(kosong)';
+    // Default depth: 3 for root (wide orientation), 1 for any subdir (focused)
+    const depth = kedalaman != null ? Math.min(Math.max(1, kedalaman), 5) : rel === '.' ? 3 : 1;
+    const listDir = (dirAbs, dirRel, d, indent, lines) => {
+        let entries;
+        try {
+            entries = readdirSync(dirAbs)
+                .filter((name) => !isProtected(dirRel === '.' ? name : `${dirRel}/${name}`, patterns))
+                .sort()
+                .map((name) => ({ name, isDir: statSync(join(dirAbs, name)).isDirectory() }));
+        }
+        catch {
+            return; // unreadable dir — skip silently
+        }
+        for (const { name, isDir } of entries) {
+            if (lines.length >= 300) {
+                lines.push('  … (dipotong — panggil lagi dengan subfolder spesifik)');
+                return;
+            }
+            lines.push(`${indent}${name}${isDir ? '/' : ''}`);
+            if (isDir && d > 1) {
+                const subAbs = join(dirAbs, name);
+                const subRel = dirRel === '.' ? name : `${dirRel}/${name}`;
+                listDir(subAbs, subRel, d - 1, `${indent}  `, lines);
+            }
+        }
+    };
+    const lines = [];
+    listDir(abs, rel, depth, '', lines);
+    return lines.length > 0 ? lines.join('\n') : '(kosong)';
 }
 /** cari — grep file contents under the working dir (case-insensitive). */
 export function cari(pola, ext) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kiosapi",
-  "version": "0.1.13",
+  "version": "0.1.17",
   "type": "module",
   "description": "CLI Kiosapi.id berbahasa Indonesia — bangun aplikasimu pakai API key Kiosapi (agen + multimodal).",
   "keywords": [