dsh-disk-cleaner 1.6.0 → 1.6.1

package/lib/index.js

@@ -349,7 +349,7 @@ export function apply(ctx) {
           additionalProperties: false,
         },
         output: {
-          schema: { type: 'object', additionalProperties: true },
+          schema: { type: 'string' },
           render: (args, value) => [{ type: 'text', text: typeof value === 'string' ? value : JSON.stringify(value, null, 2) }],
         },
         timeoutMs: 300_000,
@@ -413,6 +413,7 @@ export function apply(ctx) {
             if (ids === undefined || ids.length === 0) throw new Error('clean needs at least one target id');
             const dryRun = args.confirm !== true;
             const results = await clean(ids, { dryRun, drive: readDrive(args.drive) });
+            if (!dryRun) invalidateAudit();
             const freed = results.reduce((sum, item) => sum + item.bytes, 0);
             const files = results.reduce((sum, item) => sum + item.removed, 0);
             const failed = results.reduce((sum, item) => sum + item.failed, 0);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-disk-cleaner",
-  "version": "1.6.0",
+  "version": "1.6.1",
   "type": "module",
   "description": "Disk Cleaner for DSH: scan known-safe cache locations, see what they occupy, and reclaim the space from a panel or from the agent.",
   "keywords": [

package/smoke.mjs

@@ -260,6 +260,8 @@ check(
 check('tool needs no id for overview', JSON.stringify(definition.parameters.required) === JSON.stringify(['action']));
 check('tool description warns audit is read-only', /read-only: it never removes anything/.test(definition.description));
 check('every effect is labelled', applied.every((label) => typeof label === 'string' && label.startsWith('dsh-disk-cleaner:')));
+check('tool output schema matches its string reply', definition.output.schema.type === 'string' && typeof (await definition.execute({ action: 'overview' })) === 'string');
+check('the read-only agent clean keeps the audit cache in sync', /if \(!dryRun\) invalidateAudit\(\);[\s\S]*const freed = results\.reduce/.test(index));
 
 const ids = [...catalog.matchAll(/^\s{4}id: '([a-z-]+)',$/gm)].map((m) => m[1]);
 check('catalog has the documented 32 targets', ids.length === 32, `got ${ids.length}`);