@pitcher/cli-scripts 2.15.0 → 2.16.0

package/dist/src/commands/files.d.ts

@@ -11,6 +11,7 @@ export declare function runFilesList(opts: CommonOpts & {
     instanceId?: string;
     tags?: string;
     idIn?: string;
+    type?: string;
     page?: number;
     pageSize?: number;
 }, deps?: FilesDeps): Promise<Envelope<{

package/dist/src/commands/files.js

@@ -5,7 +5,7 @@ import { createCliError } from '../core/errors.js';
 import { emitEnvelope } from '../core/output.js';
 import { uploadFile } from '../core/upload.js';
 import { listPageParams, listPaginationEcho } from '../core/pagination.js';
-import { parseTags, parseJsonObject, validateAccessType, deepMerge, assertIntFlags } from '../schema/content.js';
+import { parseTags, parseJsonObject, validateAccessType, validateFileType, deepMerge, assertIntFlags } from '../schema/content.js';
 import { withAuthFlags } from './withAuth.js';
 export async function runFilesList(
 // NOTE: no folderId — the Django files list has no folder filter (listing a
@@ -15,12 +15,25 @@ opts, deps = {}) {
         if (!opts.instanceId)
             throw createCliError('VALIDATION', '--instance-id is required for files list');
         assertIntFlags(opts);
+        const type = validateFileType(opts.type);
         const client = clientFor(opts, deps);
         const params = { instance_id: opts.instanceId, ...listPageParams(opts) };
         if (opts.idIn)
             params.id__in = opts.idIn;
         if (opts.tags)
             params.tags__in = opts.tags;
+        // Installable apps are file records too (type=app). Exclude them by default so a "list files"
+        // request returns real content, not the instance's installed apps. `--type <content|asset|app>`
+        // isolates one type and always wins; `--type all` restores the unfiltered listing. An explicit
+        // --id-in lookup bypasses the default exclusion — fetching a file by id is a targeted lookup, not
+        // a browse, so it must return that record whatever its type (an explicit --type still applies).
+        // Uses type__in to match the tags__in/id__in CSV convention above (a single value is a valid filter).
+        if (type && type !== 'all') {
+            params.type__in = type;
+        }
+        else if (!type && !opts.idIn) {
+            params.type__in = 'content,asset';
+        }
         const res = await client.get('/api/v1/files/', { params });
         return ok({ results: res.data.results, pagination: listPaginationEcho(opts, res.data) });
     }
@@ -122,6 +135,7 @@ export function createFilesCommand() {
         .option('--instance-id <id>', 'Instance ID (required)')
         .option('--id-in <ids>', 'Comma-separated file IDs')
         .option('--tags <tags>', 'Comma-separated tags (filters via tags__in)')
+        .option('--type <type>', 'File type: content, asset, app, or all (default excludes installable apps)')
         .option('--page <n>', 'Page number', (v) => parseInt(v, 10))
         .option('--page-size <n>', 'Page size (max 500)', (v) => parseInt(v, 10))
         .action(async (options, command) => {

package/dist/src/mcp/tools/files.js

@@ -1,14 +1,21 @@
 import { z } from 'zod';
 import { runFilesList, runFilesGet, runFilesCreate, runFilesUpdate } from '../../commands/files.js';
+import { FILE_TYPES } from '../../schema/content.js';
 export const filesTools = [
     {
         name: 'pitcher_files_list',
         title: 'List files',
-        description: 'List content files in an instance. Filter by tags or specific IDs.',
+        description: 'List content files in an instance. Installed apps are stored as file records too, but are ' +
+            'EXCLUDED by default (content + assets only) so a "list files" request returns real content. ' +
+            'Pass type="app" for installed apps only, or type="all" to include everything. Filter by tags or specific IDs.',
         inputShape: {
             instanceId: z.string().describe('Instance ID (required)'),
             idIn: z.string().optional().describe('Comma-separated file IDs'),
             tags: z.string().optional().describe('Comma-separated tags'),
+            type: z
+                .enum(FILE_TYPES)
+                .optional()
+                .describe('File type filter. Omit to list content + assets (installed apps excluded); "app" = apps only; "all" = everything including apps.'),
             page: z.number().int().optional(),
             pageSize: z.number().int().optional(),
         },

package/dist/src/schema/content.d.ts

@@ -1,8 +1,10 @@
+export declare const FILE_TYPES: readonly ["content", "asset", "app", "all"];
 export declare function isPlainObject(v: unknown): v is Record<string, unknown>;
 export declare function parseJsonObject(raw: string, flagName: string): Record<string, unknown>;
 export declare function parseJsonArray(raw: string, flagName: string): unknown[];
 export declare function parseTags(raw: string): string[];
 export declare function validateAccessType(v: string | undefined): string | undefined;
+export declare function validateFileType(v: string | undefined): string | undefined;
 export declare function assertIntFlags(opts: {
     page?: number;
     pageSize?: number;

package/dist/src/schema/content.js

@@ -1,5 +1,10 @@
 import { createCliError } from '../core/errors.js';
 const ACCESS_TYPES = ['public', 'restricted', 'personal'];
+// File record types on Django's `pitcher_file.type` column, plus the `all` escape hatch.
+// Installable apps are stored as file records (type=app), so `p files list` excludes them by default;
+// `all` opts back into the unfiltered listing. See validateFileType / runFilesList.
+// Exported so the MCP `pitcher_files_list` tool sources its zod enum from the same list (no drift).
+export const FILE_TYPES = ['content', 'asset', 'app', 'all'];
 export function isPlainObject(v) {
     return typeof v === 'object' && v !== null && !Array.isArray(v);
 }
@@ -37,6 +42,14 @@ export function validateAccessType(v) {
     }
     return v;
 }
+export function validateFileType(v) {
+    if (v === undefined)
+        return undefined;
+    if (!FILE_TYPES.includes(v)) {
+        throw createCliError('VALIDATION', `--type must be one of: ${FILE_TYPES.join(', ')}`);
+    }
+    return v;
+}
 // Guards against NaN reaching the request (commander's parseInt yields NaN on
 // non-numeric input, e.g. `--page abc`).
 export function assertIntFlags(opts) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pitcher/cli-scripts",
-  "version": "2.15.0",
+  "version": "2.16.0",
   "description": "The Pitcher CLI for various tasks in its ecosystem.",
   "author": "Pitcher AG",
   "license": "MIT",