fix(node-sdk): remove broken documentParserRegistry from plugin classes

AgenticSearch, AgenticParse, and SkillPlugin were #[napi] class instances
but SessionOptions.plugins expected #[napi(object)] plain objects. napi-rs
failed to extract Option<JsDocumentParserRegistry> from null (class getter
for None), causing "Cannot convert undefined or null to object" at runtime.

Fix: simplify plugin classes to marker structs with only `kind: String`.
SkillPlugin keeps plugin_name + skills fields. The document_parser_registry
field is removed from all plugin types — plain text formats work without it
(agentic_parse falls back to fs::read_to_string), and binary format parsers
require Rust implementation not currently exposed in the Node.js SDK.

Also update DocumentParserRegistry doc to clarify it is reserved for future
use, and correct README to remove inaccurate PDF/Word/Excel claims.

Author: RoyLin

README.md

@@ -76,7 +76,7 @@ console.log(result.text);
 | Plugin          | Tool             | What it does                                                     |
 | --------------- | ---------------- | ---------------------------------------------------------------- |
 | `AgenticSearch` | `agentic_search` | Natural-language code search with IDF-weighted relevance ranking |
-| `AgenticParse`  | `agentic_parse`  | LLM-enhanced parsing for PDF, Word, CSV, code, and more          |
+| `AgenticParse`  | `agentic_parse`  | LLM-enhanced parsing for Markdown, CSV, JSON, TOML, code, and more |
 
 ```python
 from a3s_code import Agent, SessionOptions, AgenticSearch, AgenticParse

sdk/node/index.d.ts

@@ -79,6 +79,21 @@ export interface JsSecurityProvider {
 export interface JsDocumentParserRegistry {
   kind: string
 }
+/**
+ * A plugin descriptor passed in `SessionOptions.plugins`.
+ *
+ * Use the typed constructors (`new AgenticSearch()`, `new AgenticParse()`,
+ * `new SkillPlugin(...)`) to create plugin instances — do not construct this
+ * object directly.
+ */
+export interface JsPlugin {
+  /** Plugin kind: `"agentic_search"`, `"agentic_parse"`, or `"skill_plugin"`. */
+  kind: string
+  /** Plugin name (used by SkillPlugin). */
+  pluginName?: string
+  /** Skill YAML/markdown content strings (used by SkillPlugin). */
+  skills?: Array<string>
+}
 /**
  * Union type for AHP transport configuration.
  * Accepts any of: StdioTransport, HttpTransport, WebSocketTransport, UnixSocketTransport.
@@ -152,15 +167,16 @@ export interface SessionOptions {
    */
   securityProvider?: JsSecurityProvider
   /**
-   * Document parser registry for agentic_search tool.
+   * Plugin tools to mount onto this session.
+   *
+   * Pass instances of plugin classes to enable optional tools:
    *
-   * Pass `new DocumentParserRegistry()` to enable custom document format support.
-   * By default, only plain text files are searched.
    * ```js
-   * agent.session('.', { documentParserRegistry: new DocumentParserRegistry() });
+   * agent.session('.', { plugins: [new AgenticSearch()] });
+   * agent.session('.', { plugins: [new AgenticSearch(), new AgenticParse()] });
    * ```
    */
-  documentParserRegistry?: JsDocumentParserRegistry
+  plugins?: Array<JsPlugin>
   /**
    * Custom role/identity prepended before the core agentic prompt.
    * Example: "You are a senior Python developer specializing in FastAPI."
@@ -651,20 +667,77 @@ export declare class DefaultSecurityProvider {
   constructor()
 }
 /**
- * Document parser registry for agentic_search tool.
+ * Document parser registry (reserved for future use).
  *
- * Enables custom document format support (PDF, Excel, Word, etc.) for the
- * agentic_search tool. By default, only plain text files are searched.
+ * Currently only plain-text formats (source code, Markdown, JSON, TOML, YAML, CSV, etc.)
+ * are supported without additional configuration. Binary formats such as PDF, Excel, or
+ * Word require a custom `DocumentParser` implementation, which is not yet exposed in the
+ * Node.js SDK.
+ */
+export declare class DocumentParserRegistry {
+  kind: string
+  constructor()
+}
+/**
+ * Multi-phase semantic code search plugin.
+ *
+ * Mounts the `agentic_search` tool onto the session. Not registered by default.
  *
  * ```js
- * const registry = new DocumentParserRegistry();
- * agent.session('.', { documentParserRegistry: registry });
+ * agent.session('.', {
+ *   plugins: [new AgenticSearch()],
+ * });
  * ```
  */
-export declare class DocumentParserRegistry {
+export declare class AgenticSearch {
   kind: string
   constructor()
 }
+/**
+ * LLM-enhanced document parsing plugin.
+ *
+ * Mounts the `agentic_parse` tool onto the session. Not registered by default.
+ * Requires an LLM client (automatically provided from the session).
+ *
+ * ```js
+ * agent.session('.', {
+ *   plugins: [new AgenticParse()],
+ * });
+ * ```
+ */
+export declare class AgenticParse {
+  kind: string
+  constructor()
+}
+/**
+ * Skill-only plugin — injects custom skills into the session's skill registry
+ * without registering any tools.
+ *
+ * Use this to add custom LLM guidance (instructions, tool restrictions,
+ * prompting strategies) directly from Node.js. For tools, use MCP servers.
+ *
+ * ```js
+ * import { SkillPlugin } from '@a3s-lab/code';
+ *
+ * const plugin = new SkillPlugin('my-plugin', [`
+ * ---
+ * name: my-skill
+ * description: Use bash cautiously
+ * allowed-tools: "bash(*)"
+ * kind: instruction
+ * ---
+ * Always explain what command you're about to run before executing it.
+ * `]);
+ *
+ * agent.session('.', { plugins: [new AgenticSearch(), plugin] });
+ * ```
+ */
+export declare class SkillPlugin {
+  kind: string
+  pluginName?: string
+  skills?: Array<string>
+  constructor(name: string, skills: Array<string>)
+}
 /**
  * Stdio transport for AHP (Agent Harness Protocol).
  *

sdk/node/index.js

@@ -310,14 +310,17 @@ if (!nativeBinding) {
   throw new Error(`Failed to load native binding`)
 }
 
-const { EventStream, FileMemoryStore, FileSessionStore, MemorySessionStore, DefaultSecurityProvider, DocumentParserRegistry, StdioTransport, HttpTransport, WebSocketTransport, UnixSocketTransport, Agent, Session, builtinSkills, TeamTaskBoard, Team, TeamRunner, SubAgentHandle, Orchestrator } = nativeBinding
+const { EventStream, FileMemoryStore, FileSessionStore, MemorySessionStore, DefaultSecurityProvider, DocumentParserRegistry, AgenticSearch, AgenticParse, SkillPlugin, StdioTransport, HttpTransport, WebSocketTransport, UnixSocketTransport, Agent, Session, builtinSkills, TeamTaskBoard, Team, TeamRunner, SubAgentHandle, Orchestrator } = nativeBinding
 
 module.exports.EventStream = EventStream
 module.exports.FileMemoryStore = FileMemoryStore
 module.exports.FileSessionStore = FileSessionStore
 module.exports.MemorySessionStore = MemorySessionStore
 module.exports.DefaultSecurityProvider = DefaultSecurityProvider
 module.exports.DocumentParserRegistry = DocumentParserRegistry
+module.exports.AgenticSearch = AgenticSearch
+module.exports.AgenticParse = AgenticParse
+module.exports.SkillPlugin = SkillPlugin
 module.exports.StdioTransport = StdioTransport
 module.exports.HttpTransport = HttpTransport
 module.exports.WebSocketTransport = WebSocketTransport

sdk/node/src/lib.rs

@@ -473,17 +473,12 @@ impl DefaultSecurityProvider {
     }
 }
 
-/// Document parser registry.
+/// Document parser registry (reserved for future use).
 ///
-/// Enables custom document format support (PDF, Excel, Word, etc.) for
-/// plugin tools such as `AgenticSearch` and `AgenticParse`.
-///
-/// ```js
-/// const registry = new DocumentParserRegistry();
-/// agent.session('.', {
-///   plugins: [new AgenticSearch({ documentParserRegistry: registry })],
-/// });
-/// ```
+/// Currently only plain-text formats (source code, Markdown, JSON, TOML, YAML, CSV, etc.)
+/// are supported without additional configuration. Binary formats such as PDF, Excel, or
+/// Word require a custom `DocumentParser` implementation, which is not yet exposed in the
+/// Node.js SDK.
 #[napi]
 pub struct DocumentParserRegistry {
     pub kind: String,
@@ -513,22 +508,12 @@ impl DocumentParserRegistry {
 pub struct JsPlugin {
     /// Plugin kind: `"agentic_search"`, `"agentic_parse"`, or `"skill_plugin"`.
     pub kind: String,
-    /// Optional document parser registry for this plugin.
-    pub document_parser_registry: Option<JsDocumentParserRegistry>,
     /// Plugin name (used by SkillPlugin).
     pub plugin_name: Option<String>,
     /// Skill YAML/markdown content strings (used by SkillPlugin).
     pub skills: Option<Vec<String>>,
 }
 
-/// Options for the AgenticSearch plugin.
-#[napi(object)]
-#[derive(Clone, Default)]
-pub struct AgenticSearchOptions {
-    /// Pass a `DocumentParserRegistry` to enable PDF, Excel, Word, etc. support.
-    pub document_parser_registry: Option<JsDocumentParserRegistry>,
-}
-
 /// Multi-phase semantic code search plugin.
 ///
 /// Mounts the `agentic_search` tool onto the session. Not registered by default.
@@ -537,38 +522,22 @@ pub struct AgenticSearchOptions {
 /// agent.session('.', {
 ///   plugins: [new AgenticSearch()],
 /// });
-///
-/// // With document parser support:
-/// agent.session('.', {
-///   plugins: [new AgenticSearch({ documentParserRegistry: new DocumentParserRegistry() })],
-/// });
 /// ```
 #[napi]
 pub struct AgenticSearch {
     pub kind: String,
-    pub document_parser_registry: Option<JsDocumentParserRegistry>,
 }
 
 #[napi]
 impl AgenticSearch {
     #[napi(constructor)]
-    pub fn new(options: Option<AgenticSearchOptions>) -> Self {
-        let opts = options.unwrap_or_default();
+    pub fn new() -> Self {
         Self {
             kind: "agentic_search".to_string(),
-            document_parser_registry: opts.document_parser_registry,
         }
     }
 }
 
-/// Options for the AgenticParse plugin.
-#[napi(object)]
-#[derive(Clone, Default)]
-pub struct AgenticParseOptions {
-    /// Pass a `DocumentParserRegistry` to enable PDF, Excel, Word, etc. support.
-    pub document_parser_registry: Option<JsDocumentParserRegistry>,
-}
-
 /// LLM-enhanced document parsing plugin.
 ///
 /// Mounts the `agentic_parse` tool onto the session. Not registered by default.
@@ -578,28 +547,18 @@ pub struct AgenticParseOptions {
 /// agent.session('.', {
 ///   plugins: [new AgenticParse()],
 /// });
-///
-/// // With document parser support:
-/// agent.session('.', {
-///   plugins: [
-///     new AgenticParse({ documentParserRegistry: new DocumentParserRegistry() }),
-///   ],
-/// });
 /// ```
 #[napi]
 pub struct AgenticParse {
     pub kind: String,
-    pub document_parser_registry: Option<JsDocumentParserRegistry>,
 }
 
 #[napi]
 impl AgenticParse {
     #[napi(constructor)]
-    pub fn new(options: Option<AgenticParseOptions>) -> Self {
-        let opts = options.unwrap_or_default();
+    pub fn new() -> Self {
         Self {
             kind: "agentic_parse".to_string(),
-            document_parser_registry: opts.document_parser_registry,
         }
     }
 }
@@ -629,7 +588,6 @@ impl AgenticParse {
 #[napi]
 pub struct SkillPlugin {
     pub kind: String,
-    pub document_parser_registry: Option<JsDocumentParserRegistry>,
     pub plugin_name: Option<String>,
     pub skills: Option<Vec<String>>,
 }
@@ -640,7 +598,6 @@ impl SkillPlugin {
     pub fn new(name: String, skills: Vec<String>) -> Self {
         Self {
             kind: "skill_plugin".to_string(),
-            document_parser_registry: None,
             plugin_name: Some(name),
             skills: Some(skills),
         }
@@ -865,16 +822,8 @@ pub struct SessionOptions {
     /// Pass instances of plugin classes to enable optional tools:
     ///
     /// ```js
-    /// // Semantic code search
     /// agent.session('.', { plugins: [new AgenticSearch()] });
-    ///
-    /// // Document parsing with PDF support
-    /// agent.session('.', {
-    ///   plugins: [
-    ///     new AgenticSearch({ documentParserRegistry: new DocumentParserRegistry() }),
-    ///     new AgenticParse({ documentParserRegistry: new DocumentParserRegistry() }),
-    ///   ],
-    /// });
+    /// agent.session('.', { plugins: [new AgenticSearch(), new AgenticParse()] });
     /// ```
     pub plugins: Option<Vec<JsPlugin>>,
     /// Custom role/identity prepended before the core agentic prompt.
@@ -1197,12 +1146,8 @@ fn js_session_options_to_rust(options: Option<SessionOptions>) -> RustSessionOpt
             opts = opts.with_default_security();
         }
     }
-    // Mount plugins — also enable document parsing if any plugin requests it
+    // Mount plugins
     for plugin in o.plugins.iter().flatten() {
-        if plugin.document_parser_registry.is_some() && opts.document_parser_registry.is_none() {
-            opts.document_parser_registry =
-                Some(std::sync::Arc::new(a3s_code_core::DocumentParserRegistry::new()));
-        }
         match plugin.kind.as_str() {
             "agentic_search" | "agentic-search" => {
                 opts.plugins.push(std::sync::Arc::new(