feat: do not generate file if same content (#287)

closes #286

Author: reuveniy

packages/graphqlgen/src/output.ts

@@ -15,8 +15,8 @@ export function outputDefinitionFilesNotFound(
   console.log(`❌ Some path to model definitions defined in your graphqlgen.yml were not found
   
 ${chalk.bold(
-    'Step 1',
-  )}: Make sure each of these model definitions point to an existing file
+  'Step 1',
+)}: Make sure each of these model definitions point to an existing file
 
   models:
     override:
@@ -42,8 +42,8 @@ export function outputWrongSyntaxFiles(
   console.log(`❌ Some model definitions defined in your graphqlgen.yml have syntax errors
   
 ${chalk.bold(
-    'Step 1',
-  )}: Make sure each of these model definitions follow the correct syntax
+  'Step 1',
+)}: Make sure each of these model definitions follow the correct syntax
 
   ${chalk.cyan(
     `(Correct syntax: ${chalk.bold('<typeName>')}: ${chalk.bold(
@@ -74,8 +74,8 @@ export function outputInterfaceDefinitionsNotFound(
   console.log(`❌ Some model definitions defined in your graphqlgen.yml were not found
   
 ${chalk.bold(
-    'Step 1',
-  )}: Make sure each of these model definitions are defined in the following files
+  'Step 1',
+)}: Make sure each of these model definitions are defined in the following files
 
   models:
     override:
@@ -99,12 +99,12 @@ export function outputMissingModels(
   console.log(`❌ Some types from your application schema have model definitions that are not defined yet
   
 ${chalk.bold(
-    'Step 1',
-  )}: Copy/paste the model definitions below to your application
+  'Step 1',
+)}: Copy/paste the model definitions below to your application
 
 ${missingModels
-    .map(type => renderModelFromType(type, language, defaultName))
-    .join(os.EOL)}
+  .map(type => renderModelFromType(type, language, defaultName))
+  .join(os.EOL)}
 
 
 ${chalk.bold('Step 2')}: Reference the model definitions in your ${chalk.bold(
@@ -151,9 +151,9 @@ function renderTypescriptModelFromType(
   return `\
 export interface ${chalk.bold(name)} {
 ${type.fields
-    .filter(field => field.type.isScalar)
-    .map(field => `  ${field.name}: ${graphQLToTypecriptFlowType(field.type)}`)
-    .join(os.EOL)}
+  .filter(field => field.type.isScalar)
+  .map(field => `  ${field.name}: ${graphQLToTypecriptFlowType(field.type)}`)
+  .join(os.EOL)}
 }`
 }
 
@@ -165,7 +165,7 @@ function renderFlowModelFromType(
   return `\
 export interface ${chalk.bold(name)} {
 ${type.fields
-    .map(field => `  ${field.name}: ${graphQLToTypecriptFlowType(field.type)}`)
-    .join(',' + os.EOL)}
+  .map(field => `  ${field.name}: ${graphQLToTypecriptFlowType(field.type)}`)
+  .join(',' + os.EOL)}
 }`
 }

packages/graphqlgen/src/project-output.ts

@@ -1,7 +1,6 @@
 import * as Path from 'path'
 import * as FS from 'fs'
 import * as mkdirp from 'mkdirp'
-import * as rimraf from 'rimraf'
 import { GraphQLGenDefinition } from 'graphqlgen-json-schema'
 import chalk from 'chalk'
 import { CodeFileLike } from './types'
@@ -11,6 +10,18 @@ import {
 } from './path-helpers'
 import { replaceAll } from './utils'
 
+function writeChangesOnly(filename: string, content: string) {
+  if (
+    !FS.existsSync(filename) ||
+    FS.readFileSync(filename, { encoding: 'utf-8' }) !== content
+  ) {
+    console.log(chalk.green(`Overriding ${filename} with new model`))
+    FS.writeFileSync(filename, content)
+  } else {
+    console.log(chalk.gray(`File ${filename} is the same`))
+  }
+}
+
 /**
  * Bootstrap a graphqlgen config for the user to finish configuring.
  */
@@ -65,7 +76,7 @@ const writeTypes = (types: string, config: GraphQLGenDefinition): void => {
   mkdirp.sync(Path.dirname(config.output))
 
   try {
-    FS.writeFileSync(config.output, types, { encoding: 'utf-8' })
+    writeChangesOnly(config.output, types)
   } catch (e) {
     console.error(
       chalk.red(`Failed to write the file at ${config.output}, error: ${e}`),
@@ -93,13 +104,27 @@ const writeResolversScaffolding = (
 
   const outputResolversDir = config['resolver-scaffolding'].output
 
-  rimraf.sync(outputResolversDir)
+  const toBeCreatedFiles = resolvers.map(f =>
+    Path.join(outputResolversDir, f.path),
+  )
+
+  if (FS.existsSync(outputResolversDir)) {
+    FS.readdirSync(outputResolversDir)
+      .map(f => Path.join(outputResolversDir, f))
+      .filter(f => !toBeCreatedFiles.includes(f))
+      .forEach(f => {
+        FS.unlinkSync(f)
+        console.log(
+          chalk.yellow(`Deleting file ${f} - model scaffold no long available`),
+        )
+      })
+  }
 
   resolvers.forEach(f => {
     const writePath = Path.join(outputResolversDir, f.path)
     mkdirp.sync(Path.dirname(writePath))
     try {
-      FS.writeFileSync(
+      writeChangesOnly(
         writePath,
         replaceAll(
           f.code,