Enhance SDK core functionality with integration test coverage and diagnostics

TrueNine · TrueNine · commit cf8031ad905b · 2026-04-12T20:26:46.000+08:00
- Add cli-integration-test artifact and container modules
- Add runtime test coverage in cli module
- Add adaptor-core improvements for AbstractOutputAdaptor, GlobalScopeCollector, LocalizedPromptReader
- Add PromptArtifactCache, PromptCompilerDiagnostics, and PromptTypes improvements
- Update native bindings: prompt_artifact.rs, native_md_compiler transformer and serializer
- Add diagnostic_helpers and update lib exports
diff --git a/cli-integration-test/src/artifacts.ts b/cli-integration-test/src/artifacts.ts
@@ -6,16 +6,16 @@ import {fileURLToPath} from 'node:url'
 
 const REPO_ROOT = fileURLToPath(new URL('../../', import.meta.url))
 const CLI_DIR = path.join(REPO_ROOT, 'cli')
-const SCRIPT_RUNTIME_DIR = path.join(REPO_ROOT, 'libraries', 'script-runtime')
+const SDK_DIR = path.join(REPO_ROOT, 'sdk')
 const CLI_LINUX_PACKAGE_DIR = path.join(CLI_DIR, 'npm', 'linux-x64-gnu')
-const EXPECTED_LINUX_NODE_FILES = 4
+const REQUIRED_LINUX_NODE_FILE = 'napi-memory-sync-cli.linux-x64-gnu.node'
 const MAX_BUFFER = 16 * 1024 * 1024
 
 export interface CliIntegrationArtifacts {
   readonly tempDir: string
   readonly cliTarballPath: string
   readonly linuxTarballPath: string
-  readonly scriptRuntimeTarballPath: string
+  readonly sdkTarballPath: string
   readonly latestPnpmVersion: string
 }
 
@@ -93,20 +93,20 @@ function packWorkspacePackage(packageDir: string, targetDir: string): string {
 }
 
 function ensureLinuxPlatformPackageReady(): void {
-  const nodeFiles = existsSync(CLI_LINUX_PACKAGE_DIR)
-    ? readdirSync(CLI_LINUX_PACKAGE_DIR).filter(fileName => fileName.endsWith('.node'))
-    : []
+  const hasRequiredNodeFile = existsSync(CLI_LINUX_PACKAGE_DIR)
+    && readdirSync(CLI_LINUX_PACKAGE_DIR).includes(REQUIRED_LINUX_NODE_FILE)
 
-  if (nodeFiles.length >= EXPECTED_LINUX_NODE_FILES) return
+  if (hasRequiredNodeFile) return
 
   runCommand('pnpm', ['-C', CLI_DIR, 'run', 'build:napi:copy'])
 
-  const copiedNodeFiles = readdirSync(CLI_LINUX_PACKAGE_DIR)
-    .filter(fileName => fileName.endsWith('.node'))
+  const copiedNodeFiles = existsSync(CLI_LINUX_PACKAGE_DIR)
+    ? readdirSync(CLI_LINUX_PACKAGE_DIR).filter(fileName => fileName.endsWith('.node'))
+    : []
 
-  if (copiedNodeFiles.length < EXPECTED_LINUX_NODE_FILES) {
+  if (!copiedNodeFiles.includes(REQUIRED_LINUX_NODE_FILE)) {
     throw new Error(
-      `Expected ${EXPECTED_LINUX_NODE_FILES} Linux x64 NAPI artifacts in "${CLI_LINUX_PACKAGE_DIR}", found ${copiedNodeFiles.length}.`
+      `Expected the Linux x64 NAPI artifact "${REQUIRED_LINUX_NODE_FILE}" in "${CLI_LINUX_PACKAGE_DIR}", found ${copiedNodeFiles.length} .node file(s): ${copiedNodeFiles.join(', ') || '(none)'}.`
     )
   }
 }
@@ -133,17 +133,17 @@ export function prepareCliIntegrationArtifacts(): CliIntegrationArtifacts {
     CLI_LINUX_PACKAGE_DIR,
     path.join(tempDir, 'cli-linux-x64')
   )
-  const scriptRuntimeTarballPath = packWorkspacePackage(
-    SCRIPT_RUNTIME_DIR,
-    path.join(tempDir, 'script-runtime')
+  const sdkTarballPath = packWorkspacePackage(
+    SDK_DIR,
+    path.join(tempDir, 'sdk')
   )
   const latestPnpmVersion = resolveLatestPackageVersion('pnpm')
 
   cachedArtifacts = {
     tempDir,
     cliTarballPath,
     linuxTarballPath,
-    scriptRuntimeTarballPath,
+    sdkTarballPath,
     latestPnpmVersion
   }
 
diff --git a/cli-integration-test/src/container.ts b/cli-integration-test/src/container.ts
@@ -243,22 +243,22 @@ export class PreparedCliIntegrationContainer {
     this.copyPathToContainer(artifacts.cliTarballPath, containerTarballPath(artifacts.cliTarballPath))
     this.copyPathToContainer(artifacts.linuxTarballPath, containerTarballPath(artifacts.linuxTarballPath))
     this.copyPathToContainer(
-      artifacts.scriptRuntimeTarballPath,
-      containerTarballPath(artifacts.scriptRuntimeTarballPath)
+      artifacts.sdkTarballPath,
+      containerTarballPath(artifacts.sdkTarballPath)
     )
   }
 
   bootstrapLatestPnpmAndInstallCli(artifacts: CliIntegrationArtifacts): void {
     const cliTarball = containerTarballPath(artifacts.cliTarballPath)
     const linuxTarball = containerTarballPath(artifacts.linuxTarballPath)
-    const scriptRuntimeTarball = containerTarballPath(artifacts.scriptRuntimeTarballPath)
+    const sdkTarball = containerTarballPath(artifacts.sdkTarballPath)
 
     this.assertExecSuccess(
       [
         'corepack enable',
         `corepack prepare pnpm@${quoteShell(artifacts.latestPnpmVersion)} --activate`,
         'pnpm --version',
-        `pnpm add -g ${quoteShell(cliTarball)} ${quoteShell(linuxTarball)} ${quoteShell(scriptRuntimeTarball)}`,
+        `pnpm add -g ${quoteShell(cliTarball)} ${quoteShell(linuxTarball)} ${quoteShell(sdkTarball)}`,
         buildPinnedGlobalCliPlatformLinkScript(),
         'command -v tnmsc >/dev/null'
       ].join(' && ')
diff --git a/cli/src/cli-runtime.test.ts b/cli/src/cli-runtime.test.ts
@@ -1,21 +1,32 @@
 import {afterEach, describe, expect, it, vi} from 'vitest'
 
-const {cleanMock, dryRunMock, installMock, listAdaptorsMock} = vi.hoisted(() => ({
+const {
+  cleanMock,
+  dryRunMock,
+  flushOutputMock,
+  installMock,
+  listAdaptorsMock,
+  setGlobalLogLevelMock
+} = vi.hoisted(() => ({
   cleanMock: vi.fn(),
   dryRunMock: vi.fn(),
+  flushOutputMock: vi.fn(),
   installMock: vi.fn(),
-  listAdaptorsMock: vi.fn()
+  listAdaptorsMock: vi.fn(),
+  setGlobalLogLevelMock: vi.fn()
 }))
 
 vi.mock('@truenine/memory-sync-sdk', () => ({
+  flushOutput: flushOutputMock,
   getMemorySyncSdkBinding() {
     return {
       install: installMock,
       dryRun: dryRunMock,
       clean: cleanMock,
       listAdaptors: listAdaptorsMock
     }
-  }
+  },
+  setGlobalLogLevel: setGlobalLogLevelMock
 }))
 
 afterEach(() => {
diff --git a/sdk/package.json b/sdk/package.json
@@ -67,9 +67,9 @@
   },
   "devDependencies": {
     "@clack/prompts": "catalog:",
-    "@types/fs-extra": "catalog:",
     "@types/estree": "catalog:",
     "@types/estree-jsx": "catalog:",
+    "@types/fs-extra": "catalog:",
     "@types/mdast": "catalog:",
     "@types/picomatch": "catalog:",
     "@vitest/coverage-v8": "catalog:",
diff --git a/sdk/src/adaptors/adaptor-core/AbstractOutputAdaptor.ts b/sdk/src/adaptors/adaptor-core/AbstractOutputAdaptor.ts
@@ -1,4 +1,3 @@
-import type {BuildPromptTomlArtifactOptions} from '@/md-compiler'
 import type {ToolPresetName} from './GlobalScopeCollector'
 import type {RegistryWriter} from './RegistryWriter'
 import type {
@@ -32,13 +31,14 @@ import type {
   SubAgentYAMLFrontMatter,
   WslMirrorFileDeclaration
 } from './types'
+import type {BuildPromptTomlArtifactOptions} from '@/md-compiler'
 import {Buffer} from 'node:buffer'
 import * as path from 'node:path'
 import process from 'node:process'
 
+import {buildConfigDiagnostic, diagnosticLines} from '@/diagnostics'
 import {buildPromptTomlArtifact} from '@/md-compiler'
 import {buildMarkdownWithFrontMatter, buildMarkdownWithRawFrontMatter} from '@/md-compiler/markdown'
-import {buildConfigDiagnostic, diagnosticLines} from '@/diagnostics'
 import {getEffectiveHomeDir} from '@/runtime-environment'
 import {AbstractAdaptor} from './AbstractAdaptor'
 import {AdaptorKind, FilePathKind} from './enums'
diff --git a/sdk/src/adaptors/adaptor-core/GlobalScopeCollector.ts b/sdk/src/adaptors/adaptor-core/GlobalScopeCollector.ts
@@ -1,6 +1,6 @@
+import type {AdaptorOptions, UserConfigFile} from './types'
 import type {EvaluationScope} from '@/md-compiler'
 import type {CodeStylePreferences, EnvironmentContext, MdComponent, MdxGlobalScope, OsInfo, ToolReferences, UserProfile} from '@/md-compiler/globals' // Collects and manages global scope variables for MDX expression evaluation.
-import type {AdaptorOptions, UserConfigFile} from './types'
 import * as os from 'node:os'
 import process from 'node:process'
 import {OsKind, ShellKind, ToolPresets} from '@/md-compiler/globals'
diff --git a/sdk/src/adaptors/adaptor-core/LocalizedPromptReader.ts b/sdk/src/adaptors/adaptor-core/LocalizedPromptReader.ts
@@ -1,4 +1,3 @@
-import type {MdxGlobalScope} from '@/md-compiler/globals'
 import type {PromptCompilerDiagnosticContext} from './PromptCompilerDiagnostics'
 import type {
   DirectoryReadResult,
@@ -13,6 +12,7 @@ import type {
   PromptKind,
   ReadError
 } from './types'
+import type {MdxGlobalScope} from '@/md-compiler/globals'
 import {
   buildDiagnostic,
   buildFileOperationDiagnostic,
diff --git a/sdk/src/adaptors/adaptor-core/PromptArtifactCache.ts b/sdk/src/adaptors/adaptor-core/PromptArtifactCache.ts
@@ -2,9 +2,9 @@ import type {MdxGlobalScope} from '@/md-compiler/globals'
 import type {ParsedMarkdown} from '@/md-compiler/markdown'
 import * as fs from 'node:fs'
 import * as path from 'node:path'
+import JSON5 from 'json5'
 import {mdxToMd} from '@/md-compiler'
 import {parseMarkdown} from '@/md-compiler/markdown'
-import JSON5 from 'json5'
 
 export interface PromptArtifact {
   readonly rawMdx: string
diff --git a/sdk/src/adaptors/adaptor-core/PromptCompilerDiagnostics.test.ts b/sdk/src/adaptors/adaptor-core/PromptCompilerDiagnostics.test.ts
@@ -1,6 +1,6 @@
 import * as path from 'node:path'
-import {UndefinedNamespaceError} from '@/md-compiler/errors'
 import {describe, expect, it} from 'vitest'
+import {UndefinedNamespaceError} from '@/md-compiler/errors'
 import {
   formatPromptCompilerDiagnostic,
   resolveSourcePathForDistFile
diff --git a/sdk/src/adaptors/adaptor-core/PromptTypes.ts b/sdk/src/adaptors/adaptor-core/PromptTypes.ts
@@ -1,6 +1,6 @@
-import type {Root, RootContent} from '@/md-compiler'
 import type {ClaudeCodeCLISubAgentColors, CodingAgentTools, FilePathKind, NamingCaseKind, PromptKind, RuleScope} from './enums'
 import type {GlobalConfigDirectory} from './OutputTypes'
+import type {Root, RootContent} from '@/md-compiler'
 
 /** Common directory representation */
 export interface Path<K extends FilePathKind = FilePathKind> {
diff --git a/sdk/src/adaptors/adaptor-core/plugin.ts b/sdk/src/adaptors/adaptor-core/plugin.ts
@@ -1,4 +1,3 @@
-import type {MdxGlobalScope} from '@/md-compiler/globals'
 import type {
   AindexConfig,
   CodeStylesOptions,
@@ -13,6 +12,7 @@ import type {InputCollectedContext, OutputCollectedContext} from './InputTypes'
 import type {NativeDeskPathsBinding} from '@/core/desk-paths-types'
 import type {ExecutionPlan} from '@/execution-plan'
 import type {ILogger} from '@/libraries/logger'
+import type {MdxGlobalScope} from '@/md-compiler/globals'
 import type {RuntimeCommand} from '@/runtime-command'
 import {Buffer} from 'node:buffer'
 import * as fs from 'node:fs'
diff --git a/sdk/src/core/input_plugins/prompt_artifact.rs b/sdk/src/core/input_plugins/prompt_artifact.rs
@@ -1,5 +1,5 @@
-use serde_json::Value;
 use crate::md_compiler::{MdxGlobalScope, MdxToMdOptions, mdx_to_md_with_metadata};
+use serde_json::Value;
 
 #[derive(Debug, Clone)]
 pub struct PromptArtifact {
diff --git a/sdk/src/diagnostic_helpers.rs b/sdk/src/diagnostic_helpers.rs
@@ -1,5 +1,5 @@
-use serde_json::{Map, Value};
 use crate::logger::LoggerDiagnosticInput;
+use serde_json::{Map, Value};
 
 pub(crate) fn line(value: impl Into<String>) -> Vec<String> {
   vec![value.into()]
diff --git a/sdk/src/index.ts b/sdk/src/index.ts
@@ -181,51 +181,66 @@ export type {
   LoggerDiagnosticRecord,
   LogLevel
 } from './libraries/logger'
+export {
+  defineProxy,
+  getProxyModuleConfig,
+  loadProxyModule,
+  resolvePublicPath,
+  resolvePublicPathUnchecked,
+  validatePublicPath
+} from './libraries/script-runtime'
+export type {
+  ProxyCommand,
+  ProxyContext,
+  ProxyDefinition,
+  ProxyMatcherConfig,
+  ProxyModule,
+  ProxyModuleConfig,
+  ProxyRouteHandler,
+  ValidatePublicPathOptions
+} from './libraries/script-runtime'
 export {
   buildPromptTomlArtifact,
   buildTomlDocument,
   mdxToMd
 } from './md-compiler'
-export {
-  buildFrontMatter,
-  buildMarkdownWithFrontMatter,
-  buildMarkdownWithRawFrontMatter,
-  buildRawFrontMatter,
-  doubleQuoted,
-  parseMarkdown,
-  transformMdxReferencesToMd,
-  wrapRawFrontMatter
-} from './md-compiler/markdown'
-export {
-  CompilerDiagnosticError,
-  ExportParseError,
-  ScopeError,
-  UndefinedNamespaceError,
-  UndefinedVariableError,
-  createCompilerDiagnostic,
-  formatCompilerDiagnostic
-} from './md-compiler/errors'
-export {
-  OsKind,
-  ShellKind,
-  ToolPresets
-} from './md-compiler/globals'
 export type {
   BuildPromptTomlArtifactOptions,
   BuildTomlDocumentOptions,
   EvaluationScope,
   ExportMetadata,
   MdxFlowExpression,
   MdxGlobalScope,
+  MdxjsEsm,
   MdxJsxFlowElement,
   MdxJsxTextElement,
   MdxTextExpression,
   MdxToMdOptions,
   MdxToMdResult,
-  MdxjsEsm,
   Root,
   RootContent
 } from './md-compiler'
+export {
+  CompilerDiagnosticError,
+  createCompilerDiagnostic,
+  ExportParseError,
+  formatCompilerDiagnostic,
+  ScopeError,
+  UndefinedNamespaceError,
+  UndefinedVariableError
+} from './md-compiler/errors'
+export type {
+  CompilerDiagnostic,
+  CompilerDiagnosticContext,
+  CompilerDiagnosticPoint,
+  CompilerDiagnosticPosition,
+  FormatCompilerDiagnosticOptions
+} from './md-compiler/errors'
+export {
+  OsKind,
+  ShellKind,
+  ToolPresets
+} from './md-compiler/globals'
 export type {
   CodeStylePreferences,
   EnvironmentContext,
@@ -234,35 +249,20 @@ export type {
   ToolReferences,
   UserProfile
 } from './md-compiler/globals'
-export type {
-  CompilerDiagnostic,
-  CompilerDiagnosticContext,
-  CompilerDiagnosticPoint,
-  CompilerDiagnosticPosition,
-  FormatCompilerDiagnosticOptions
-} from './md-compiler/errors'
+export {
+  buildFrontMatter,
+  buildMarkdownWithFrontMatter,
+  buildMarkdownWithRawFrontMatter,
+  buildRawFrontMatter,
+  doubleQuoted,
+  parseMarkdown,
+  transformMdxReferencesToMd,
+  wrapRawFrontMatter
+} from './md-compiler/markdown'
 export type {
   BuildMarkdownOptions,
   ParsedMarkdown
 } from './md-compiler/markdown'
-export {
-  defineProxy,
-  getProxyModuleConfig,
-  loadProxyModule,
-  resolvePublicPath,
-  resolvePublicPathUnchecked,
-  validatePublicPath
-} from './libraries/script-runtime'
-export type {
-  ProxyCommand,
-  ProxyContext,
-  ProxyDefinition,
-  ProxyMatcherConfig,
-  ProxyModule,
-  ProxyModuleConfig,
-  ProxyRouteHandler,
-  ValidatePublicPathOptions
-} from './libraries/script-runtime'
 export type {
   ListPromptsOptions,
   ManagedPromptKind,
diff --git a/sdk/src/inputs/AbstractInputCapability.ts b/sdk/src/inputs/AbstractInputCapability.ts
@@ -1,4 +1,3 @@
-import type {ParsedMarkdown} from '@/md-compiler/markdown'
 import type {
   AdaptorOptions,
   AdaptorScopeRegistration,
@@ -12,13 +11,14 @@ import type {
   ResolvedBasePaths,
   YAMLFrontMatter
 } from '@/adaptors/adaptor-core'
+import type {ParsedMarkdown} from '@/md-compiler/markdown'
 import {spawn} from 'node:child_process'
 import * as path from 'node:path'
 
-import {parseMarkdown} from '@/md-compiler/markdown'
 import {PathPlaceholders} from '@/adaptors/adaptor-core'
 import {buildDiagnostic, diagnosticLines} from '@/diagnostics'
 import {createLogger} from '@/libraries/logger'
+import {parseMarkdown} from '@/md-compiler/markdown'
 import {logProtectedDeletionGuardError, ProtectedDeletionGuardError} from '@/ProtectedDeletionGuard'
 import {resolveUserPath} from '@/runtime-environment'
 
diff --git a/sdk/src/inputs/runtime.ts b/sdk/src/inputs/runtime.ts
diff --git a/sdk/src/internal/prompts-legacy.ts b/sdk/src/internal/prompts-legacy.ts
diff --git a/sdk/src/lib.rs b/sdk/src/lib.rs
diff --git a/sdk/src/md-compiler/index.ts b/sdk/src/md-compiler/index.ts
diff --git a/sdk/src/native_md_compiler/serializer.rs b/sdk/src/native_md_compiler/serializer.rs
diff --git a/sdk/src/native_md_compiler/transformer.rs b/sdk/src/native_md_compiler/transformer.rs
