chore(msar): Issue tracking

battis · Jan 3, 2025 · 4ea54aa · 4ea54aa
1 parent 9cf0079
commit 4ea54aa
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/packages/msar/src/bin/commands/download.ts b/packages/msar/src/bin/commands/download.ts
@@ -32,6 +32,11 @@ import * as Snapshot from '../../workflows/Snapshot.js';
 
   const snapshotPath = path.resolve(process.cwd(), snapshotPathArg!);
 
+  /*
+   * TODO abstract as much of bin/download into workflow/Download as possible
+   *   The basic design is that the bin scripts are just UI wrappers around
+   *   real objects, not _part_ of the objects
+   */
   if (!outputPath) {
     outputPath = path.join(
       path.dirname(snapshotPath!),
@@ -113,6 +118,7 @@ import * as Snapshot from '../../workflows/Snapshot.js';
     'index.json'
   );
   await common.output.writeJSON(indexPath, index, { pretty });
+  // TODO output metadata.json of CLI arguments for downloads too
 
   if (quit) {
     await spider.quit();

diff --git a/packages/msar/src/workflows/Download/Downloader/AuthenticatedFetch.ts b/packages/msar/src/workflows/Download/Downloader/AuthenticatedFetch.ts
@@ -39,6 +39,7 @@ export class Downloader
     outputOptions: { outputPath },
     ...options
   }: Options) {
+    // FIXME AuthenticatedFetch is instantiating headless
     super(`https://${host}`, options);
     if (!outputPath) {
       throw new common.output.OutputError(