track-cli 4.1.0 → 4.2.0-rc2

package/esm/src/action/add.js

@@ -1,6 +1,6 @@
 import * as dntShim from "../../_dnt.shims.js";
 import { CONFIG_FILE_PATH, load as loadConfig } from "../shared/config.js";
-import { CantReadFile, NewFilesNotAllowed } from "../shared/errors.js";
+import { CantReadFile, NewFilesNotAllowed, RejectNewFilesInTestFolder, RejectTooLongFilepath } from "../shared/errors.js";
 import { pathsToFilenameSet, toNativeStyle } from "../shared/file.js";
 import { getCommonChallengeContext, listFileNames, printWorkingFileSet, trackClientFromConfig, } from "../shared/mod.js";
 import { FileListType } from "../shared/types.js";
@@ -20,6 +20,17 @@ export async function add(filePaths) {
     // Don't allow the config file to be added
     existingPaths.add(CONFIG_FILE_PATH);
     const pathsToAdd = Array.from(requestedPaths).filter((p) => !existingPaths.has(p));
+    // Files in test folder are not allowed
+    const filesInTestFolder = pathsToAdd.filter(v => v.startsWith("test/"));
+    if (filesInTestFolder.length > 0) {
+        throw new RejectNewFilesInTestFolder(filesInTestFolder);
+    }
+    // Reject long filepath
+    // https://givery.slack.com/archives/C05RABF6KU3/p1772069298650949
+    const longFilepaths = pathsToAdd.filter(v => v.length > 150);
+    if (longFilepaths.length > 0) {
+        throw new RejectTooLongFilepath(longFilepaths);
+    }
     const addedFiles = listFileNames(codingContext, FileListType.UserAddedFiles)
         .concat(pathsToAdd);
     const updatedFiles = {};

package/esm/src/action/run.js

@@ -8,7 +8,6 @@ import { EnvSettings, FileListType, } from "../shared/types.js";
 import * as colors from "../../deps/deno.land/std@0.195.0/fmt/colors.js";
 import { exists } from "../../deps/deno.land/std@0.195.0/fs/mod.js";
 import { FileNotFound } from "../shared/errors.js";
-const MAX_MEMORY_FOR_TEST = 512 * 1024 * 1024; // 512 MB
 export async function run(options) {
     const config = await loadConfig();
     const api = trackClientFromConfig(config);
@@ -26,6 +25,7 @@ export async function run(options) {
         cacheDirs: envConfig.cacheDirs,
         username: envConfig.username,
         baseDir: envConfig.baseDir,
+        keepContainer: true,
     });
     const envVars = Object.assign({}, envConfig.variables);
     envVars["CHALLENGE_LANGUAGE"] = codingContext.challengeLanguage;
@@ -91,7 +91,6 @@ export async function run(options) {
             files: buildRan ? [] : files,
             envVars: {},
             tarballUrls: buildRan ? [] : codingContext.tarballUrls,
-            memoryBytes: MAX_MEMORY_FOR_TEST,
         };
         const scoreCounter = new CountPassingLineHandler();
         const runDoneData = await orca.simpleRunCommand(runCmd, scoreCounter);

package/esm/src/meta.d.ts

@@ -1,3 +1,3 @@
-export declare const VERSION = "4.1.0";
+export declare const VERSION = "4.2.0-rc2";
 export declare const DESCRIPTION = "A CLI for interacting with tracks.run and running code tests on track's servers";
 export declare function checkUpdates(): Promise<boolean>;

package/esm/src/meta.js

@@ -1,7 +1,7 @@
 import * as colors from "../deps/deno.land/std@0.195.0/fmt/colors.js";
 // @ts-ignore: has no exported member
 import { CookieJar, fetch } from "node-fetch-cookies";
-export const VERSION = "4.1.0";
+export const VERSION = "4.2.0-rc2";
 export const DESCRIPTION = "A CLI for interacting with tracks.run and running code tests on track's servers";
 const VERSION_RE = /^(\d+)\.(\d+)\.(\d+)(?:-?(.*))$/;
 function parseSemver(s) {

package/esm/src/orca/types.d.ts

@@ -15,6 +15,7 @@ export type PrepareCommand = {
     cacheDirs: string[];
     username?: string;
     baseDir?: string;
+    keepContainer?: boolean;
 };
 export type SimpleRunCommand = {
     id: string;

package/esm/src/shared/errors.d.ts

@@ -79,6 +79,14 @@ export declare class FileMissingOrOutsideDirectory extends TrackError {
 export declare class NewFilesNotAllowed extends TrackError {
     constructor();
 }
+export declare class RejectNewFilesInTestFolder extends TrackError {
+    fileNames: string[];
+    constructor(fileNames: string[]);
+}
+export declare class RejectTooLongFilepath extends TrackError {
+    fileNames: string[];
+    constructor(fileNames: string[]);
+}
 export declare class DeleteFilesNotAllowed extends TrackError {
     constructor();
 }

package/esm/src/shared/errors.js

@@ -222,6 +222,30 @@ export class NewFilesNotAllowed extends TrackError {
         this.name = "NewFilesNotAllowed";
     }
 }
+export class RejectNewFilesInTestFolder extends TrackError {
+    constructor(fileNames) {
+        super(`New files under test folder are not allowed: ${fileNames.join(", ")}`);
+        Object.defineProperty(this, "fileNames", {
+            enumerable: true,
+            configurable: true,
+            writable: true,
+            value: fileNames
+        });
+        this.name = "RejectNewFilesInTestFolder";
+    }
+}
+export class RejectTooLongFilepath extends TrackError {
+    constructor(fileNames) {
+        super(`More than 150 characters in filepath are not allowed: ${fileNames.join(", ")}`);
+        Object.defineProperty(this, "fileNames", {
+            enumerable: true,
+            configurable: true,
+            writable: true,
+            value: fileNames
+        });
+        this.name = "RejectTooLongFilepath";
+    }
+}
 export class DeleteFilesNotAllowed extends TrackError {
     constructor() {
         super("This challenge does not allow the deletion of files");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "track-cli",
-  "version": "4.1.0",
+  "version": "4.2.0-rc2",
   "description": "A CLI for interacting with tracks.run and running code tests on track's servers",
   "repository": {
     "type": "git",