refactor: use ensureNetlifyIgnore from @netlify/dev-utils (#7448)

* Replaces usages of ensureNetlifyIgnore with dev-utils

* Corrected import group

* Uninstalled parse-gitignore packages

* Added back in gitignore util to track a logger

Author: keanu-a

package-lock.json

@@ -136,7 +136,6 @@
     "p-wait-for": "5.0.2",
     "parallel-transform": "1.2.0",
     "parse-github-url": "1.0.3",
-    "parse-gitignore": "2.0.0",
     "prettyjson": "1.2.5",
     "raw-body": "3.0.0",
     "read-package-up": "11.0.0",
@@ -181,7 +180,6 @@
     "@types/node-fetch": "2.6.12",
     "@types/parallel-transform": "1.1.4",
     "@types/parse-github-url": "1.0.3",
-    "@types/parse-gitignore": "1.0.2",
     "@types/picomatch": "4.0.1",
     "@types/prettyjson": "0.0.33",
     "@types/semver": "7.7.0",

package.json

@@ -7,8 +7,8 @@ import type { NetlifyAPI } from '@netlify/api'
 import { listSites } from '../../lib/api.js'
 import { startSpinner } from '../../lib/spinner.js'
 import { chalk, logAndThrowError, exit, log, APIError, netlifyCommand } from '../../utils/command-helpers.js'
-import getRepoData from '../../utils/get-repo-data.js'
 import { ensureNetlifyIgnore } from '../../utils/gitignore.js'
+import getRepoData from '../../utils/get-repo-data.js'
 import { track } from '../../utils/telemetry/index.js'
 import type { SiteInfo } from '../../utils/types.js'
 import BaseCommand from '../base-command.js'

src/commands/link/link.ts

@@ -1,41 +1,6 @@
-import { readFile, writeFile } from 'fs/promises'
-import path from 'path'
+import { ensureNetlifyIgnore as originalEnsureNetlifyIgnore } from '@netlify/dev-utils'
 
-import parseIgnore from 'parse-gitignore'
+import { log, warn, logError } from './command-helpers.js'
 
-import { fileExistsAsync } from '../lib/fs.js'
-
-import { log } from './command-helpers.js'
-
-const hasGitIgnore = async function (dir: string) {
-  const gitIgnorePath = path.join(dir, '.gitignore')
-  const hasIgnore = await fileExistsAsync(gitIgnorePath)
-  return hasIgnore
-}
-
-export const ensureNetlifyIgnore = async function (dir: string) {
-  const gitIgnorePath = path.join(dir, '.gitignore')
-  const ignoreContent = '# Local Netlify folder\n.netlify\n'
-
-  /* No .gitignore file. Create one and ignore .netlify folder */
-  if (!(await hasGitIgnore(dir))) {
-    await writeFile(gitIgnorePath, ignoreContent, 'utf8')
-    return false
-  }
-
-  let gitIgnoreContents
-  let ignorePatterns
-  try {
-    gitIgnoreContents = await readFile(gitIgnorePath, 'utf8')
-    ignorePatterns = parseIgnore.parse(gitIgnoreContents)
-  } catch {
-    // ignore
-  }
-  /* Not ignoring .netlify folder. Add to .gitignore */
-  if (!ignorePatterns || !ignorePatterns.patterns.some((pattern) => /(^|\/|\\)\.netlify($|\/|\\)/.test(pattern))) {
-    log()
-    log('Adding local .netlify folder to .gitignore file...')
-    const newContents = `${gitIgnoreContents}\n${ignoreContent}`
-    await writeFile(gitIgnorePath, newContents, 'utf8')
-  }
-}
+export const ensureNetlifyIgnore = async (rootDir: string) =>
+  originalEnsureNetlifyIgnore(rootDir, { log, warn, error: logError })