@peng272/dsh-wechat-ilink 0.7.0 → 0.7.2

package/CHANGELOG.md

@@ -15,6 +15,45 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 > documentation this project has: every one of them was an assumed DSH contract
 > that turned out to be wrong.
 
+## [0.7.2]
+
+### Fixed
+
+- **The startup banner reported the wrong version.** `PLUGIN_VERSION` was a
+  hardcoded literal mirrored from `package.json`, and it drifted on the first
+  release that bumped the version: 0.7.1 was loaded while the banner said 0.7.0.
+  Since the banner is deliberately the way a loaded build is identified, that
+  defeats its purpose. The version is now read from this package's own
+  `package.json` at load, so it cannot drift again.
+## [0.7.1]
+
+### Fixed
+
+- **A fresh conversation could never start.** Every turn passed
+  `--session-id <sessionId>`, but `dsh --profile headless --session-id` requires
+  that session to already exist. On a new install the recorded id
+  (`wechat-clawbot`) does not exist yet, so **every** inbound message failed with
+
+  ```
+  dsh: session "wechat-clawbot" does not exist; omit --session-id to start a new Session
+  ```
+
+  and produced no reply. The first turn now omits the flag (which creates the
+  session), the id the driver reports is persisted to `<stateDir>/session.json`,
+  and later turns resume it. A recorded session that has since disappeared is
+  detected and transparently restarted.
+
+  Found by reading the channel log of a live install; the failure was silent from
+  WeChat's side (the message simply got no answer).
+
+- Driver-level failures emitted as a `{"type":"error"}` frame are now surfaced in
+  the log instead of being reported only as `reason=incomplete`.
+
+### Changed
+
+- The package is published as **`@peng272/dsh-wechat-ilink`**. The bare name
+  `dsh-wechat-ilink` belongs to another maintainer and cannot be updated from
+  this account.
 ## [0.7.0]
 
 A reimplementation of the channel internals. The **runtime identities are

package/lib/index.js

@@ -16,6 +16,7 @@ import fs from 'node:fs'
 import path from 'node:path'
 import { spawn } from 'node:child_process'
 import { randomUUID } from 'node:crypto'
+import { fileURLToPath } from 'node:url'
 
 import { AccountStore, resolveStateDir } from './accounts.js'
 import { ILinkClient, QrLoginManager, DEFAULT_BASE_URL, renderQrPng } from './ilink.js'
@@ -24,8 +25,25 @@ import { Config } from './schema.js'
 
 export const name = 'wechat-clawbot'
 
-/** Plugin version, mirrored from package.json (a literal keeps the bundle JSON-free). */
-export const PLUGIN_VERSION = '0.7.0'
+/**
+ * Plugin version, read from this package's own `package.json`.
+ *
+ * This used to be a hardcoded literal mirrored from package.json, and it drifted
+ * on the very first release that bumped the version — the banner advertised
+ * 0.7.0 while 0.7.1 was loaded, which defeats the point of having a banner.
+ * Reading the file costs one synchronous read at load and cannot drift.
+ */
+function readOwnVersion() {
+  try {
+    const here = path.dirname(fileURLToPath(import.meta.url))
+    const pkg = JSON.parse(fs.readFileSync(path.join(here, '..', 'package.json'), 'utf-8'))
+    return typeof pkg.version === 'string' && pkg.version ? pkg.version : '0.0.0'
+  } catch {
+    return '0.0.0'
+  }
+}
+
+export const PLUGIN_VERSION = readOwnVersion()
 
 /**
  * Services this plugin requires before it activates.
@@ -102,6 +120,7 @@ export function parseHeadlessJson(stdout) {
   let sessionId
   let text = ''
   let reason
+  let directError
   for (const line of String(stdout ?? '').split('\n')) {
     const trimmed = line.trim()
     if (!trimmed) continue
@@ -114,12 +133,15 @@ export function parseHeadlessJson(stdout) {
     if (event.type === 'session' && typeof event.sessionId === 'string') sessionId = event.sessionId
     else if (event.type === 'final' && typeof event.text === 'string') text = event.text
     else if (event.type === 'status' && event.phase === 'turn_end' && event.reason) reason = event.reason
+    // A driver-level failure (for example an unknown --session-id) is emitted as
+    // its own frame, not as a turn_end reason.
+    else if (event.type === 'error' && typeof event.message === 'string') directError = event.message
   }
   return {
     sessionId,
     text,
     reason: reason ? reason.kind : 'incomplete',
-    error: reason?.error?.message,
+    error: reason?.error?.message ?? directError,
   }
 }
 
@@ -215,9 +237,39 @@ export function apply(ctx, rawConfig) {
   let sessionId = config.sessionId?.trim() || 'wechat-clawbot'
   let chain = Promise.resolve()
 
-  function runHeadless(prompt) {
-    if (!dshPrefix) return Promise.resolve({ text: '', reason: 'no-dsh-prefix' })
-    const argv = [...dshPrefix, '--profile', 'headless', '--json', '--session-id', sessionId, prompt]
+  // `dsh --profile headless --session-id <id>` requires that session to ALREADY
+  // exist. Passing it for a fresh conversation fails every single turn with
+  //   session "<id>" does not exist; omit --session-id to start a new Session
+  // so the first run omits the flag and the id it reports is remembered for the
+  // turns after it. A recorded session that has since disappeared is detected
+  // from stderr and retried without the flag.
+  const sessionStateFile = path.join(stateDir, 'session.json')
+  let sessionEstablished = false
+  try {
+    if (fs.existsSync(sessionStateFile)) {
+      const saved = JSON.parse(fs.readFileSync(sessionStateFile, 'utf-8'))
+      if (typeof saved.id === 'string' && saved.id) sessionId = saved.id
+      sessionEstablished = saved.established === true
+    }
+  } catch {
+    /* start a fresh conversation */
+  }
+
+  function rememberSession(id, established) {
+    if (id) sessionId = id
+    sessionEstablished = established
+    try {
+      fs.mkdirSync(stateDir, { recursive: true })
+      fs.writeFileSync(sessionStateFile, JSON.stringify({ id: sessionId, established }, null, 2), 'utf-8')
+    } catch (error) {
+      log('error', `could not persist session state: ${error?.message ?? error}`)
+    }
+  }
+
+  function spawnHeadless(prompt, useSessionId) {
+    const argv = [...dshPrefix, '--profile', 'headless', '--json']
+    if (useSessionId) argv.push('--session-id', sessionId)
+    argv.push(prompt)
     return new Promise((resolve) => {
       let child
       try {
@@ -256,11 +308,23 @@ export function apply(ctx, rawConfig) {
         if (parsed.reason === 'incomplete' && stderr.trim()) {
           log('error', `headless exited code=${code}: ${stderr.trim().slice(0, 400)}`)
         }
-        finish(parsed)
+        finish({ ...parsed, stderr, exitCode: code })
       })
     })
   }
 
+  async function runHeadless(prompt) {
+    if (!dshPrefix) return { text: '', reason: 'no-dsh-prefix' }
+    let result = await spawnHeadless(prompt, sessionEstablished)
+    if (result.reason === 'incomplete' && /does not exist|omit --session-id/i.test(result.stderr || '')) {
+      log('error', 'the recorded session is gone; starting a new conversation')
+      rememberSession(sessionId, false)
+      result = await spawnHeadless(prompt, false)
+    }
+    if (result.sessionId) rememberSession(result.sessionId, true)
+    return result
+  }
+
   // ---- inbound ----
   function extractText(msg) {
     const items = Array.isArray(msg?.item_list) ? msg.item_list : []

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@peng272/dsh-wechat-ilink",
-  "version": "0.7.0",
+  "version": "0.7.2",
   "description": "WeChat ClawBot (Tencent iLink) channel for DeepSeek Harness 0.2.0-rc.2 — talk to DSH from WeChat.",
   "type": "module",
   "main": "lib/index.js",