Add checks for mutually-exclusive sf orders ls flags

.github/workflows/ci.yml

@@ -9,3 +9,7 @@ jobs:
 
     steps:
       - uses: actions/checkout@v2
+      - uses: denoland/setup-deno@v2
+      - run: deno install
+      - run: deno fmt --check
+      - run: deno check ./src/index.ts

README.md

@@ -31,7 +31,9 @@ sf --version  # 0.1.0
 - Test changes with
   - `deno run devv` to test against local API
   - `deno run prod` to test against production API
-  - The `deno run <env>` is an alias to the user facing `sf` command. So if you wanted to run `sf login` locally against the local API, run `deno run devv login`
+  - The `deno run <env>` is an alias to the user facing `sf` command. So if you
+    wanted to run `sf login` locally against the local API, run
+    `deno run devv login`
 
 ## New Release
 

deno.json

@@ -4,5 +4,11 @@
   },
   "imports": {
     "@std/assert": "jsr:@std/assert@1"
+  },
+  "lint": {
+    "exclude": ["src/schema.ts"]
+  },
+  "fmt": {
+    "exclude": ["src/schema.ts"]
   }
 }

package.json

@@ -45,4 +45,4 @@
     "typescript": "^5.6.2"
   },
   "version": "0.1.35"
-}
+}

src/checkVersion.ts

@@ -68,7 +68,7 @@ async function checkProductionCLIVersion() {
   // Fetch from network
   try {
     const response = await fetch(
-      "https://raw.githubusercontent.com/sfcompute/cli/refs/heads/main/package.json"
+      "https://raw.githubusercontent.com/sfcompute/cli/refs/heads/main/package.json",
     );
     if (!response.ok) {
       throw new Error(`HTTP error! status: ${response.status}`);
@@ -102,7 +102,7 @@ export async function checkVersion() {
 
   if (isPatchUpdate) {
     console.log(
-      chalk.cyan(`Automatically upgrading ${version} → ${latestVersion}`)
+      chalk.cyan(`Automatically upgrading ${version} → ${latestVersion}`),
     );
     try {
       execSync("sf upgrade", { stdio: "inherit" });
@@ -130,7 +130,7 @@ Run 'sf upgrade' to update to the latest version
         padding: 1,
         borderColor: "yellow",
         borderStyle: "round",
-      })
+      }),
     );
   }
 }

src/helpers/config.ts

@@ -27,7 +27,7 @@ const ConfigDefaults = process.env.IS_DEVELOPMENT_CLI_ENV
 // --
 
 export async function saveConfig(
-  config: Partial<Config>
+  config: Partial<Config>,
 ): Promise<{ success: boolean }> {
   const configPath = getConfigPath();
   const configDir = join(homedir(), ".sfcompute");

src/helpers/errors.ts

@@ -16,13 +16,13 @@ export function logLoginMessageAndQuit(): never {
 export async function logSessionTokenExpiredAndQuit(): Promise<never> {
   await clearAuthFromConfig();
   logAndQuit(
-    `\nYour session has expired. Please login again.\n\n\t$ ${loginCommand}\n`
+    `\nYour session has expired. Please login again.\n\n\t$ ${loginCommand}\n`,
   );
 }
 
 export function failedToConnect(): never {
   logAndQuit(
-    "Failed to connect to the server. Please check your internet connection and try again."
+    "Failed to connect to the server. Please check your internet connection and try again.",
   );
 }
 

src/helpers/price.ts

@@ -4,7 +4,7 @@ export function pricePerGPUHourToTotalPriceCents(
   pricePerGPUHourCents: Cents,
   durationSeconds: number,
   nodes: number,
-  gpusPerNode: number
+  gpusPerNode: number,
 ): Cents {
   const totalGPUs = nodes * gpusPerNode;
   const totalHours = durationSeconds / 3600;
@@ -16,7 +16,7 @@ export function totalPriceToPricePerGPUHour(
   priceCents: number,
   durationSeconds: number,
   nodes: number,
-  gpusPerNode: number
+  gpusPerNode: number,
 ): Cents {
   const totalGPUs = nodes * gpusPerNode;
   const totalHours = durationSeconds / 3600;

src/helpers/units.ts

@@ -29,10 +29,11 @@ export function roundStartDate(startDate: Date): Date {
 
 export function computeApproximateDurationSeconds(
   startDate: Date | "NOW",
-  endDate: Date
+  endDate: Date,
 ): number {
-  const startEpoch =
-    startDate === "NOW" ? currentEpoch() : dateToEpoch(startDate);
+  const startEpoch = startDate === "NOW"
+    ? currentEpoch()
+    : dateToEpoch(startDate);
   const endEpoch = dateToEpoch(endDate);
   return dayjs(epochToDate(endEpoch)).diff(dayjs(epochToDate(startEpoch)), "s");
 }
@@ -57,7 +58,7 @@ interface PriceWholeToCentsReturn {
   invalid: boolean;
 }
 export function priceWholeToCents(
-  price: string | number
+  price: string | number,
 ): PriceWholeToCentsReturn {
   if (
     price === null ||

src/helpers/urls.ts

@@ -38,7 +38,7 @@ const apiPaths = {
 
 export async function getWebAppUrl(
   key: keyof typeof webPaths,
-  params?: any
+  params?: any,
 ): Promise<string> {
   const config = await loadConfig();
   const path = webPaths[key];
@@ -51,7 +51,7 @@ export async function getWebAppUrl(
 
 export async function getApiUrl(
   key: keyof typeof apiPaths,
-  params?: any
+  params?: any,
 ): Promise<string> {
   const config = await loadConfig();
   const path = apiPaths[key];

src/helpers/waitingForOrder.ts

@@ -5,7 +5,7 @@ import { getOrder } from "./fetchers.ts";
 
 export async function waitForOrderToNotBePending(orderId: string) {
   const spinner = ora(
-    `Order ${orderId} - pending (this can take a moment)`
+    `Order ${orderId} - pending (this can take a moment)`,
   ).start();
 
   // 1 minute
@@ -18,7 +18,7 @@ export async function waitForOrderToNotBePending(orderId: string) {
       spinner.succeed();
       return order;
     }
-    await new Promise(resolve => setTimeout(resolve, 500));
+    await new Promise((resolve) => setTimeout(resolve, 500));
   }
 
   spinner.fail();

src/index.ts

@@ -14,7 +14,7 @@ import { registerDev } from "./lib/dev.ts";
 import { registerLogin } from "./lib/login.ts";
 import { registerMe } from "./lib/me.ts";
 import { registerOrders } from "./lib/orders/index.tsx";
-import { IS_TRACKING_DISABLED, analytics } from "./lib/posthog.ts";
+import { analytics, IS_TRACKING_DISABLED } from "./lib/posthog.ts";
 import { registerSell } from "./lib/sell.ts";
 import { registerTokens } from "./lib/tokens.ts";
 import { registerScale } from "./lib/updown.tsx";
@@ -68,7 +68,7 @@ const main = async () => {
       }
     }
 
-    program.exitOverride(error => {
+    program.exitOverride((error) => {
       let isError = true;
 
       switch (error.code) {
@@ -91,8 +91,8 @@ const main = async () => {
           const nextArg = arr[i + 1];
           if (nextArg && !nextArg.startsWith("-")) {
             (acc as Record<string, string | number | boolean>)[key] = isNaN(
-              Number(nextArg)
-            )
+                Number(nextArg),
+              )
               ? nextArg
               : Number(nextArg);
           } else {
@@ -103,7 +103,9 @@ const main = async () => {
       }, {});
 
       analytics.track({
-        event: `${process.argv[2] || "unknown"}${process.argv[3] ? "_" + process.argv[3] : ""}`,
+        event: `${process.argv[2] || "unknown"}${
+          process.argv[3] ? "_" + process.argv[3] : ""
+        }`,
         properties: {
           ...args,
           shell: process.env.SHELL,

src/lib/ConfirmInput.tsx

@@ -26,7 +26,7 @@ const ConfirmInput: React.FC<ConfirmInputProps> = ({
     (newValue: string) => {
       onSubmit(yn(newValue, { default: isChecked }));
     },
-    [isChecked, onSubmit]
+    [isChecked, onSubmit],
   );
 
   return (

src/lib/Quote.tsx

@@ -40,17 +40,17 @@ export default function QuoteDisplay(props: { quote: Quote }) {
 
 export type Quote =
   | {
-      price: number;
-      quantity: number;
-      start_at: string;
-      end_at: string;
-      instance_type: string;
-    }
+    price: number;
+    quantity: number;
+    start_at: string;
+    end_at: string;
+    instance_type: string;
+  }
   | {
-      price: number;
-      quantity: number;
-      start_at: string;
-      end_at: string;
-      contract_id: string;
-    }
+    price: number;
+    quantity: number;
+    start_at: string;
+    end_at: string;
+    contract_id: string;
+  }
   | null;

src/lib/balance.ts

@@ -20,7 +20,7 @@ export function registerBalance(program: Command) {
     .command("balance")
     .description("Get account balance")
     .option("--json", "Output in JSON format")
-    .action(async options => {
+    .action(async (options) => {
       const {
         available: { whole: availableWhole, cents: availableCents },
         reserved: { whole: reservedWhole, cents: reservedCents },
@@ -57,7 +57,7 @@ export function registerBalance(program: Command) {
             "Reserved",
             chalk.gray(formattedReserved),
             chalk.gray(reservedCents.toLocaleString()),
-          ]
+          ],
         );
 
         console.log(table.toString() + "\n");
@@ -91,7 +91,7 @@ export async function getBalance(): Promise<BalanceUsdCents> {
 
   if (!data) {
     return logAndQuit(
-      `Failed to get balance: Unexpected response from server: ${response}`
+      `Failed to get balance: Unexpected response from server: ${response}`,
     );
   }