work on docs, tests, etc.

Author: toptobes

.env.example

@@ -20,3 +20,6 @@ ASTRA_EXE_PATH=build/native/nativeCompile/astra
 # Required for lifecycle tests: Cloud provider/region where resources will be created
 ASTRA_CLOUD=
 ASTRA_REGION=
+
+# Set to true to enable running (potentially long running and costly) PCU lifecycle tests
+ASTRA_RUN_PCU_LIFECYCLE_TESTS=false

.idea/dictionaries/project.xml

@@ -351,7 +351,23 @@ This may unfortunately require some changes to your scripts if you are relying o
 <details>
   <summary><strong>CSV schema changes</strong></summary>
 
-  TODO
+  Previously,the CSV output of commands would be somewhat inconsistent, or even invalid CSV in some cases (bugs).
+
+  Now, every command supporting CSV output will return an RFC-4180-compliant response of exactly this schema:
+
+  ```csv
+  code,message,data_field_1,data_field_2,...
+  ```
+
+  The `code` field will be of type `"OK" | "UNCAUGHT" | "DATABASE_NOT_FOUND" | "..."`, and if no rows are present, an implicit OK code should be assumed.
+ 
+  The `message` field will be a human-friendly message describing the result or error of the command (if applicable), and will be empty if there is nothing to say.
+
+  The remaining fields will be the relevant data fields for the command, and may be empty if there is no data to show.
+
+  The `code` and `message` fields may be repeated for each row of data, to keep the CSV schema consistent.
+
+  Note that values _may_ be wrapped in quotes as necessary to ensure valid CSV output, as per the RFC-4180 specification.
 </details>
 
 ### Consistent flag names and behavior

.idea/inspectionProfiles/Project_Default.xml

@@ -29,7 +29,7 @@ dependencies {
     implementation("org.slf4j:slf4j-nop:2.0.17")
 
     // for underlying api calls
-    implementation("com.datastax.astra:astra-db-java:2.0.0")
+    implementation("com.datastax.astra:astra-db-java:2.1.0")
 //    implementation("com.datastax.astra:astra-sdk-devops:1.2.9")
     implementation(files("astra-sdk-devops.jar")) // temporary until it's on maven central
 

.idea/workspace.xml

@@ -27,7 +27,7 @@ com.datastax.astra.client.tables.definition.rows.Row
 com.datastax.astra.client.tables.definition.columns.*
 com.datastax.astra.client.tables.definition.*
 
-# Pcu gateways (TODO replace w/ SDK paths when available)
+# Pcu gateways
 com.dtsx.astra.cli.gateways.pcu.vendored.domain.*
 com.dtsx.astra.sdk.pcu.domain.*
 

README.md

@@ -1,7 +1,5 @@
 #!/usr/bin/env sh
 
-# TODO detect if you are migrating from v0.x
-
 set -eu
 IFS=$(printf '\n\t')
 
@@ -188,9 +186,9 @@ while [ "$AUTO_YES_INSTALL" = 0 ]; do
       echo ""
       echo "${RED}To use a custom installation path, please globally set the ASTRA_HOME environment variable.${RESET}"
       echo ""
-      echo "This variable $(tput bold)must remain in place forever${RESET} (e.g. in your shell profile like $(underline "~/.bashrc"), $(underline "~/.zshrc"), etc.) so that the CLI can always locate its home directory."
+      echo "This variable $(tput bold)must remain in place forever${RESET} (e.g. in your shell profile like $(underline "~/.zprofile"), $(underline "~/.bash_profile"), etc.) so that the CLI can always locate its home directory."
       echo ""
-      echo "After setting it, restart your terminal or run 'source ~/.bashrc' (or the appropriate file) to apply the change."
+      echo "After setting it, restart your terminal or run 'source ~/.zprofile' (or the appropriate file) to apply the change."
       exit 1
       ;;
     [Cc]* )

build.gradle.kts

@@ -135,7 +135,7 @@ public static void main(String... args) {
             CliPropertiesImpl.mkAndLoadSysProps(cliEnv, MemoizedCliProperties::new),
             OutputType.HUMAN,
             new AstraColors(Ansi.AUTO),
-            new AstraLogger(Level.REGULAR, getCtx, false, Optional.empty(), true),
+            new AstraLogger(Level.REGULAR, getCtx, false, Optional.empty(), Optional.empty()),
             new AstraConsole(System.in, mkPrintWriter(System.out, "stdout"), mkPrintWriter(System.err, "stderr"), null, getCtx, false),
             new AstraHome(getCtx),
             FileSystems.getDefault(),

reflected.txt

@@ -129,7 +129,7 @@ public final void run() {
             ctx.fs(),
             ctx.gateways(),
             ctx.upgradeNotifier(),
-            ctx.forceUseProfile()
+            ctx.forceProfileForTesting()
         ));
     }