fix #5

src/commands/auth.ts

@@ -23,7 +23,7 @@ export default async (props: Props, env: SystemEnvironment, authServer: AuthServ
     try {
       token = await authServer.requestAuthToken()
     } catch (e) {
-      out.write(couldNotRetrieveTokenMessage)
+      out.writeError(couldNotRetrieveTokenMessage)
       process.exit(1)
     }
 

src/commands/console.ts

@@ -15,7 +15,7 @@ export default async (props: Props, env: SystemEnvironment): Promise<void> => {
   const currentProjectId = readProjectIdFromProjectFile(resolver)
 
   if (!currentProjectId) {
-    out.write(noProjectFileMessageFound)
+    out.writeError(noProjectFileMessageFound)
     process.exit(1)
   }
 

src/commands/export.ts

@@ -19,7 +19,7 @@ export default async(props: Props, env: SystemEnvironment): Promise<void> => {
 
     const projectId = props.projectId ? props.projectId : readProjectIdFromProjectFile(resolver)
     if (!projectId) {
-      out.write(noProjectIdMessage)
+      out.writeError(noProjectIdMessage)
       process.exit(0)
     }
 
@@ -37,7 +37,7 @@ export default async(props: Props, env: SystemEnvironment): Promise<void> => {
     if (e.errors) {
       const errors = parseErrors(e)
       const output = generateErrorOutput(errors)
-      out.write(`${output}`)
+      out.writeError(`${output}`)
     } else {
       throw e
     }

src/commands/init.ts

@@ -29,7 +29,7 @@ export default async (props: Props, env: SystemEnvironment): Promise<void> => {
   const {resolver, out} = env
 
   if (resolver.exists(graphcoolProjectFileName) && resolver.read(graphcoolProjectFileName).toString().includes('# projectId:')) {
-    out.write(projectAlreadyExistsMessage)
+    out.writeError(projectAlreadyExistsMessage)
     process.exit(1)
   }
 
@@ -57,12 +57,12 @@ export default async (props: Props, env: SystemEnvironment): Promise<void> => {
   } catch (e) {
     out.stopSpinner()
     debug(`Could not create project: ${JSON.stringify(e)}`)
-    out.write(`${couldNotCreateProjectMessage}`)
+    out.writeError(`${couldNotCreateProjectMessage}`)
 
     if (e.errors) {
       const errors = parseErrors(e)
       const output = generateErrorOutput(errors)
-      out.write(`${output}`)
+      out.writeError(`${output}`)
     } else {
       throw e
     }

src/commands/projects.ts

@@ -41,7 +41,7 @@ export default async (props: Props, env: SystemEnvironment): Promise<void> => {
     // out.write(outputString)
     out.write(output)
   } catch (e) {
-    out.write(`${couldNotFetchProjectsMessage} ${e.message}`)
+    out.writeError(`${couldNotFetchProjectsMessage} ${e.message}`)
     process.exit(1)
   }
 

src/commands/pull.ts

@@ -21,7 +21,7 @@ export default async (props: Props, env: SystemEnvironment): Promise<void> => {
     const projectId = props.projectId ? props.projectId : readProjectIdFromProjectFile(resolver)
 
     if (!projectId) {
-      out.write(noProjectIdMessage)
+      out.writeError(noProjectIdMessage)
       process.exit(1)
     }
 
@@ -43,7 +43,7 @@ export default async (props: Props, env: SystemEnvironment): Promise<void> => {
     if (e.errors) {
       const errors = parseErrors(e)
       const output = generateErrorOutput(errors)
-      out.write(`${output}`)
+      out.writeError(`${output}`)
     } else {
       throw e
     }

src/commands/push.ts

@@ -26,13 +26,13 @@ export default async (props: Props, env: SystemEnvironment): Promise<void> => {
   const {resolver, out} = env
 
   if (!resolver.exists(graphcoolProjectFileName) && !resolver.exists(`${props.projectFilePath}/${graphcoolProjectFileName}`)) {
-    out.write(noProjectFileMessage)
+    out.writeError(noProjectFileMessage)
     process.exit(1)
   }
 
   const projectInfo = readProjectInfoFromProjectFile(resolver, props.projectFilePath)
   if (!projectInfo) {
-    out.write(invalidProjectFileMessage)
+    out.writeError(invalidProjectFileMessage)
     process.exit(1)
   }
 
@@ -71,20 +71,20 @@ export default async (props: Props, env: SystemEnvironment): Promise<void> => {
       }
     }
 
-    // something went wrong
+    // can't do migration because of issues with schema
     else if (migrationResult.messages.length === 0 && migrationResult.errors.length > 0) {
       out.write(`\n${migrationErrorMessage}`)
       printMigrationErrors(migrationResult.errors, out)
     }
 
   } catch (e) {
     out.stopSpinner()
-    out.write(couldNotMigrateSchemaMessage)
+    out.writeError(couldNotMigrateSchemaMessage)
 
     if (e.errors) {
       const errors = parseErrors(e)
       const output = generateErrorOutput(errors)
-      out.write(`${output}`)
+      out.writeError(`${output}`)
     } else {
       throw e
     }

src/system/StdOut.ts

@@ -9,6 +9,10 @@ export default class StdOut implements Out {
     process.stdout.write(message)
   }
 
+  writeError(message: string): void {
+    process.stderr.write(message)
+  }
+
   startSpinner(message: string) {
     this.spinner = ora(message).start()
   }

src/system/TestOut.ts

@@ -5,6 +5,9 @@ export default class TestOut implements Out {
   write(message: string): void {
   }
 
+  writeError(message: string): void {
+  }
+
   startSpinner(message: string) {
   }
 

src/types.ts

@@ -66,6 +66,7 @@ export interface MigrationResult {
 
 export interface Out {
   write(message: string): void
+  writeError(message: string): void
   startSpinner(message: string): void
   stopSpinner(): void
 }