openclaw - 💡(How to fix) Fix [CI] build-smoke/check regressions in compaction, model registry, and skills config [1 comments, 1 participants]

src/agents/compaction.ts(242,11): error TS2345: Argument of type 'AbortSignal' is not assignable to parameter of type 'Record<string, string>'. src/agents/pi-extensions/compaction-safeguard.ts(617,44): error TS2339: Property 'getApiKey' does not exist on type 'ModelRegistry'. src/agents/skills/config.ts(53,42): error TS2339: Property 'source' does not exist on type 'Skill'.

Fix Plan

To resolve the type and build regressions, we need to address the errors in the affected files. Here are the steps:

• Update the compaction.ts file to correctly assign the AbortSignal type.
• Add the missing getApiKey property to the ModelRegistry type in compaction-safeguard.ts.
• Add the missing source property to the Skill type in config.ts.

Example code changes:

// src/agents/compaction.ts
import { AbortSignal } from 'abort-controller';

// Update the function parameter type
function compactData(signal: AbortSignal) {
  // ...
}

// src/agents/pi-extensions/compaction-safeguard.ts
interface ModelRegistry {
  // Add the missing property
  getApiKey: () => string;
}

// src/agents/skills/config.ts
interface Skill {
  // Add the missing property
  source: string;
}

Verification

To verify the fix, re-run the failed jobs in the CI pipeline and check for any remaining errors. The build-smoke, build-artifacts, check, check-additional, checks-fast-contracts-protocol, and checks-fast-extensions jobs should all pass.

Extra Tips

• Make sure to update the type definitions for any dependent modules or files.
• Run npm run build and npm run check locally to catch any type errors before pushing changes to the CI pipeline.
• Consider adding additional type checks or tests to prevent similar regressions in the future.