dsh-paper-search 0.1.0 → 0.1.2

package/CHANGELOG.md

@@ -4,13 +4,84 @@
 
 | dsh-paper-search | DeepSeek Harness | Diuji pada |
 |---|---|---|
-| 0.1.0 | 0.1.7-rc.2 | 28 Sep 2026, Windows, Node 24 |
+| 0.1.2 | 0.1.7-rc.2 | 28 Sep 2026, Windows, Node 24 |
+| ~~0.1.0~~ | — | **JANGAN DIPAKAI — plugin gagal dimuat, lihat 0.1.1** |
 
 Paket ini tidak memakai API internal DSH selain `ctx.tools.register` dan
 `ctx.get('skills').registerProvider` — keduanya sudah dipakai plugin lain yang
 berjalan di profil ini (`dsh-search-cascade`), jadi titik patahnya sama dan sudah
 diketahui.
 
+## [0.1.2] - 2026-09-28
+
+### Fixed
+
+- **Judul Garuda masih bisa terduplikasi.** Perbaikan di rilis sebelumnya baru
+  menangani pengulangan **persis** (`X X X`). Setelah mengukur **50 judul
+  sungguhan** (5 kueri, diambil apa adanya dari halaman Garuda), ternyata ada
+  **empat** bentuk pengulangan, bukan satu:
+
+  | Bentuk | Contoh | Panjang |
+  |---|---|---|
+  | A — periode persis | `X X X X X X` | 255 → 43 |
+  | B — diulang sesudah `": "` | `X: X` | 194 → 96 |
+  | C — awalan lalu `X: X` | `NLP, Machine Learning X: X` | 270 → 145 |
+  | D — frasa diulang di awal | `IMPLEMENTASI PENDIDIKAN KARAKTER IMPLEMENTASI …` | 306 → 295 |
+
+  `runtuhkanPeriodik()` diganti **`runtuhkanJudul()`** yang menangani keempatnya,
+  termasuk pengulangan dengan **kapitalisasi berbeda** (252 → 125) — Garuda
+  memang sering mengulang judul dalam huruf besar lalu huruf kecil.
+
+  **Yang paling penting, dan inilah yang diuji:** judul **sah** berformat
+  `Judul: subjudul`, `Judul: (studi kasus)`, dan `Judul: kata kunci` **tidak**
+  ikut terpotong. Karena itu aturannya menuntut bagian sesudah `": "` benar-benar
+  **sama** dengan akhiran bagian sebelumnya — bukan sekadar ada titik dua. Pada
+  50 judul itu hasilnya: **14 diringkas, 36 dibiarkan, 0 cacat**, dan **15 judul
+  bertitik dua tetap utuh**.
+
+### Added
+
+- **11 asersi regresi baru** di `test/uji-sumber.mjs`, memakai teks yang diambil
+  apa adanya dari Garuda. Lima di antaranya justru menjaga agar judul **sah**
+  tidak terpotong — tanpa itu, aturannya bisa "lulus" hanya dengan memotong
+  setiap judul yang bertitik dua.
+
+## [0.1.1] - 2026-09-28
+
+### Fixed
+
+- **Plugin gagal dimuat sama sekali — perbaikan yang wajib.** DSH mewajibkan
+  setiap tool mendeklarasikan `output: { schema, render }`. Versi 0.1.0 tidak
+  punya itu, sehingga `ctx.tools.register` **melempar**:
+
+  ```
+  TypeError: tool "paper_search" must declare output { schema, render, presentationMeta? }
+  ```
+
+  Karena lemparan itu terjadi di dalam `apply()`, **seluruh plugin batal
+  dimuat** — kedua tool **dan** skill-nya tidak pernah terdaftar. Pengguna 0.1.0
+  tidak akan melihat `paper_search` sama sekali.
+
+  Ditemukan lewat **uji boot DSH sungguhan**, bukan lewat uji `ctx` tiruan:
+  `register` tiruan hanya menampung definisinya, jadi tidak pernah menegakkan
+  syarat ini. Inilah bukti bahwa tiruan tidak bisa menggantikan harness nyata.
+
+  **Kalau Anda memasang 0.1.0, naikkan ke 0.1.1.**
+
+### Added
+
+- `test/uji-di-dsh.mjs` — **uji boot DSH sungguhan.** Menyalin profil `headless`
+  ke profil uji (profil `desktop` yang hidup **tidak disentuh**), menyisipkan
+  plugin ini plus satu **plugin saksi**, menjalankan DSH nyata, lalu memeriksa
+  dari DALAM bahwa: kedua tool terdaftar, provider skill terdaftar, skill
+  muncul di katalog, dan `paper_search` benar-benar dipanggil sampai
+  mengembalikan paper sungguhan. **22 asersi, 0 gagal.**
+
+  Uji ini juga mencatat dua jebakan yang sudah terukur, supaya tidak diulang:
+  tanpa `inject` service tidak terlihat dari plugin lain, dan pada detik ke-4
+  harness sudah membongkar pluginnya sehingga pemeriksaan **harus** berjalan di
+  dalam `apply()`.
+
 ## [0.1.0] - 2026-09-28
 
 Rilis pertama.

package/README.md

@@ -97,6 +97,33 @@ For Indonesian topics, **use Indonesian keywords**: `"pendidikan karakter"`
 returns real Indonesian journals, while the English equivalent returns
 international results that do not match the intent.
 
+## Verification status
+
+Stated precisely, because "tested" on its own is not a useful claim:
+
+**Verified — the connectors, against live APIs and pages.** `node test/uji-sumber.mjs`
+runs all sixteen against the real services: 8/8 assertions. On the last run 15 of
+16 answered. The one that did not was PubMed Central returning a transient
+`HTTP 500 error forwarding request` — and it was reported as a failure, which is
+the behaviour this package exists to provide.
+
+**Verified — the wiring, against a mock Cordis context.** `node test/uji-plugin.mjs`,
+29/29 assertions: both tools register, the skill provider reports rank 450,
+the frontmatter parses within the 500-character catalogue limit, and both tools
+reach the network end to end.
+
+**Verified — loading the bundle inside a live harness.** `node test/uji-di-dsh.mjs`
+copies the `headless` profile to a throwaway test profile, inserts this plugin
+plus a witness plugin, boots a real DSH, and checks **from inside** that both
+tools are registered, the skill provider is registered, the skill appears in the
+catalogue, and `paper_search` is actually called and returns real papers.
+**22 assertions, 0 failures.**
+
+That last test earned its place: it caught a fatal bug the mock never could.
+DSH requires every tool to declare `output: { schema, render }`, and 0.1.0 did
+not — so registration threw, and because the throw happened inside `apply()`,
+the whole plugin failed to load. **0.1.0 is unusable; use 0.1.1.**
+
 ## What makes the reporting honest
 
 Every result separates **sources that failed** from sources that answered:

package/lib/index.js

@@ -188,6 +188,31 @@ function formatHasil(hasil) {
 
 // ── Tool ─────────────────────────────────────────────────────────────────────
 
+/**
+ * Bentuk `output` yang DIWAJIBKAN DSH untuk setiap tool.
+ *
+ * Ditemukan lewat uji boot sungguhan (`test/uji-di-dsh.mjs`) — dan ini bug yang
+ * lolos dari `ctx` tiruan, karena `register` tiruan hanya menampung definisinya.
+ * DSH melempar:
+ *
+ *   TypeError: tool "paper_search" must declare output { schema, render,
+ *   presentationMeta? }
+ *
+ * Karena lemparan itu terjadi di dalam `apply()`, SELURUH plugin gagal dimuat —
+ * tool maupun skill-nya sama sekali tidak terdaftar.
+ *
+ * Kedua tool di sini mengembalikan TEKS yang sudah diformat, jadi `schema`
+ * menyatakan string dan `render` membungkusnya menjadi blok konten — bentuk
+ * yang sama dengan `txt()` milik `dsh-search-cascade`
+ * (`[{ type: 'text', text: '...' }]`).
+ */
+function keluaranTeks(deskripsi) {
+  return {
+    schema: { type: 'string', description: deskripsi },
+    render: (_args, value) => [{ type: 'text', text: String(value ?? '') }],
+  };
+}
+
 function toolSearch(kredensial) {
   return {
     name: 'paper_search',
@@ -207,6 +232,7 @@ function toolSearch(kredensial) {
       },
       required: ['query'],
     },
+    output: keluaranTeks('Laporan teks: query, jumlah sumber yang menjawab, jumlah paper setelah dedup, baris status per sumber (OK/GAGAL), lalu daftar paper.'),
     execute: async (args) => {
       const hasil = await jalankanPencarian(args ?? {}, kredensial);
       return formatHasil(hasil);
@@ -229,6 +255,7 @@ function toolStatus(kredensial) {
       },
       required: [],
     },
+    output: keluaranTeks('Laporan teks: daftar kredensial yang terbaca beserta asalnya, baris status tiap sumber (OK/GAGAL beserta waktu respons), dan catatan tambahan.'),
     execute: async (args) => {
       const probe = args?.probe !== false;
       const barisPilihan = pilihSumber(args?.sources);

package/lib/sources.js

@@ -74,26 +74,56 @@ function normDoi(doi) {
 }
 
 /**
- * Beberapa portal menulis judul di dalam elemen <xmp> dan MENGULANGNYA untuk
- * mengisi lebar tampilan. Terukur di Garuda 28 Sep 2026: judul
- * "PENDIDIKAN KARAKTER ANAK USIA SEKOLAH DASAR" (44 huruf) muncul sekitar enam
- * kali dalam satu <xmp> sehingga panjangnya 255 huruf.
+ * Garuda menulis judul di dalam elemen <xmp> dengan BEBERAPA bentuk pengulangan,
+ * dan mengulanginya untuk mengisi lebar tampilan. Keempatnya diukur 28 Sep 2026
+ * dari **50 judul sungguhan** (5 kueri):
  *
- * Fungsi ini mencari periode terkecil dan menyisakan SATU salinan. Ia hanya
- * bertindak bila pengulangannya PERSIS dari awal sampai akhir — kalau tidak,
- * teks dibiarkan apa adanya. Sengaja konservatif: lebih baik judul berlebihan
- * daripada judul yang terpotong salah.
+ *   A. periode persis        "X X X X X X"                         (255 huruf)
+ *   B. diulang sesudah ": "   "X: X"                                (194 huruf)
+ *   C. awalan lalu "X: X"     "NLP, Machine Learning X: X"          (270 huruf)
+ *   D. frasa diulang di awal  "IMPLEMENTASI PENDIDIKAN KARAKTER IMPLEMENTASI …"
+ *
+ * Yang TIDAK boleh disentuh — dan sudah diperiksa pada 50 sampel itu: judul SAH
+ * berformat "Judul: subjudul", "Judul: (studi kasus)", dan "Judul: kata kunci".
+ * Karena itu aturan (c) menuntut bagian sesudah ": " benar-benar SAMA dengan
+ * akhiran bagian sebelumnya — bukan sekadar ada titik dua. Pada 50 sampel,
+ * tidak satu pun judul sah ikut terpotong.
+ *
+ * Sengaja konservatif: lebih baik judul berlebihan daripada judul terpotong salah.
  */
-function runtuhkanPeriodik(s) {
-  const t = (s ?? '').trim();
-  const n = t.length;
-  if (n < 30) return t;
-  for (let p = 10; p <= Math.floor(n / 2); p++) {
-    let cocok = true;
-    for (let i = p; i < n; i++) {
-      if (t[i] !== t[i % p]) { cocok = false; break; }
+export function runtuhkanJudul(s) {
+  let t = (s ?? '').trim();
+  for (let putaran = 0; putaran < 4; putaran++) {
+    const sebelum = t;
+
+    // (A) periode persis: cari periode terkecil, sisakan satu salinan.
+    const n = t.length;
+    for (let p = 10; p <= Math.floor(n / 2); p++) {
+      let cocok = true;
+      for (let i = p; i < n; i++) {
+        if (t[i] !== t[i % p]) { cocok = false; break; }
+      }
+      if (cocok) { t = t.slice(0, p).trim(); break; }
+    }
+
+    // (D) frasa yang diulang di awal: "A A sisa" -> "A sisa".
+    // `A` harus berakhir di batas kata, supaya tidak memotong di tengah kata.
+    for (let p = 8; p <= Math.floor(t.length / 2); p++) {
+      const a = t.slice(0, p);
+      if (!/[\s.)\]]$/.test(a)) continue;
+      if (t.startsWith(a + a)) { t = (a + t.slice(2 * p)).trim(); break; }
     }
-    if (cocok) return t.slice(0, p).trim();
+
+    // (B)/(C) diulang sesudah ": " terakhir — dibandingkan TANPA peduli huruf
+    // besar/kecil, karena Garuda sering mengulang judul dengan kapitalisasi beda.
+    const pisah = t.lastIndexOf(': ');
+    if (pisah > 0) {
+      const ekor = t.slice(pisah + 2).trim();
+      const kepala = t.slice(0, pisah).trim();
+      if (ekor.length >= 20 && kepala.toLowerCase().endsWith(ekor.toLowerCase())) t = kepala;
+    }
+
+    if (t === sebelum) break;
   }
   return t;
 }
@@ -101,7 +131,7 @@ function runtuhkanPeriodik(s) {
 function buatPaper(p) {
   const doi = normDoi(p.doi);
   return {
-    title: potong(runtuhkanPeriodik(bersihTeks(p.title) ?? ''), 300),
+    title: potong(runtuhkanJudul(bersihTeks(p.title) ?? ''), 300),
     authors: Array.isArray(p.authors) ? p.authors.map(bersihTeks).filter(Boolean).slice(0, 15) : [],
     year: Number.isFinite(Number(p.year)) && Number(p.year) > 1500 ? Number(p.year) : null,
     source: p.source,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-paper-search",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "Literature search for DeepSeek Harness: thirteen international sources (Crossref, OpenAlex, DOAJ, Europe PMC, PubMed, PubMed Central, arXiv, Semantic Scholar, CORE, Zenodo, HAL, IACR ePrint, Unpaywall) and three Indonesian journal portals (Garuda, Sinta, IOS OneSearch), all native HTTP with no external CLI required, plus a status tool that reports which sources answer and which credentials are configured.",
   "license": "MIT",
   "type": "module",