revert: roll back PR review fixes per maintainer call

- TonSignData: drop strictCheckDeviceSupport (set to false) and remove
  getVersionRange(). Firmware version that ships TonSignData isn't decided
  yet; gating now would lock out every connected device. Re-enable both
  once the touch/classic1s min versions are known.

- LedgerConnectorBase.call: restore JSON.stringify(params) in debugLog.
  Log bus only feeds the host's local defaultLogger pipeline (no third-party
  telemetry forwarding), so full params are acceptable for now and useful
  for in-the-wild debugging.

Author: ByteZhang1024

packages/core/src/api/ton/TonSignData.ts

@@ -9,7 +9,10 @@ import type { TonSignDataParams } from '../../types/api/tonSignData';
 
 export default class TonSignData extends BaseMethod<HardwareTonSignData> {
   init() {
-    this.strictCheckDeviceSupport = true;
+    // Keep strict-check off until the firmware release that ships TonSignData
+    // is decided — we don't yet know the min versions for touch/classic1s.
+    // Flip back to true and add getVersionRange() once those numbers land.
+    this.strictCheckDeviceSupport = false;
     this.checkDeviceId = true;
     this.allowDeviceMode = [...this.allowDeviceMode, UI_REQUEST.NOT_INITIALIZE];
 
@@ -47,22 +50,6 @@ export default class TonSignData extends BaseMethod<HardwareTonSignData> {
     };
   }
 
-  // Required because strictCheckDeviceSupport=true above. Without an entry
-  // here, BaseMethod returns an empty range and the core treats every
-  // device as unsupported, blocking the call before run() ever fires.
-  // Aligned with TonSignMessage's getFixInitStateErrorVersionRange() — bump
-  // when the firmware release that ships TonSignData lands.
-  getVersionRange() {
-    return {
-      model_touch: {
-        min: '4.13.0',
-      },
-      model_classic1s: {
-        min: '3.12.0',
-      },
-    };
-  }
-
   async run() {
     const res = await this.device.commands.typedCall('TonSignData', 'TonSignedData', {
       ...this.params,

packages/hwk-ledger-adapter/src/connector/LedgerConnectorBase.ts

@@ -88,30 +88,6 @@ const METHOD_PREFIX_TO_APP_NAME: Record<string, string> = {
   tron: 'Tron',
 };
 
-/**
- * Render call params as non-sensitive metadata for the SDK log bus. Raw
- * payloads (serializedTx / psbt / rawTxHex / messageHex / typed data) MUST
- * NOT enter the log stream — hosts may forward it to telemetry / persistent
- * stores. We surface only path, length-of-payload, and a few harmless flags.
- */
-function summarizeCallParams(_method: string, params: unknown): string {
-  if (!params || typeof params !== 'object') return '';
-  const p = params as Record<string, unknown>;
-  const safe: Record<string, unknown> = {};
-  if (typeof p.path === 'string') safe.path = p.path;
-  if (typeof p.showOnDevice === 'boolean') safe.showOnDevice = p.showOnDevice;
-  if (typeof p.checkOnDevice === 'boolean') safe.checkOnDevice = p.checkOnDevice;
-  for (const key of ['serializedTx', 'rawTxHex', 'messageHex', 'message', 'psbt', 'data']) {
-    const v = p[key];
-    if (typeof v === 'string') safe[`${key}.len`] = v.length;
-    else if (v !== undefined) safe[`${key}.type`] = typeof v;
-  }
-  if (Array.isArray(p.tokenSignatures)) {
-    safe.tokenSignaturesCount = p.tokenSignatures.length;
-  }
-  return JSON.stringify(safe);
-}
-
 // ---------------------------------------------------------------------------
 // Default signer kit importer (webpack/rspack — uses "exports" field)
 // ---------------------------------------------------------------------------
@@ -405,7 +381,7 @@ export class LedgerConnectorBase implements IConnector {
   // ---------------------------------------------------------------------------
 
   async call(sessionId: string, method: string, params: unknown): Promise<unknown> {
-    debugLog('[DMK] call:', method, summarizeCallParams(method, params));
+    debugLog('[DMK] call:', method, JSON.stringify(params));
     // Bind the chain's Ledger app name to ctx.wrapError once per dispatch so
     // chain handlers can call `ctx.wrapError(err)` with no per-site appName.
     const ctx = this._ctxForMethod(method);