netlify-cli 10.17.2 → 10.17.5

package/npm-shrinkwrap.json

@@ -1,21 +1,21 @@
 {
   "name": "netlify-cli",
-  "version": "10.17.2",
+  "version": "10.17.5",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "netlify-cli",
-      "version": "10.17.2",
+      "version": "10.17.5",
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {
-        "@netlify/build": "^27.11.3",
-        "@netlify/config": "^18.1.3",
+        "@netlify/build": "^27.11.4",
+        "@netlify/config": "^18.1.4",
         "@netlify/edge-bundler": "^1.12.1",
         "@netlify/framework-info": "^9.2.0",
         "@netlify/local-functions-proxy": "^1.1.1",
-        "@netlify/plugins-list": "^6.38.0",
+        "@netlify/plugins-list": "^6.39.0",
         "@netlify/zip-it-and-ship-it": "^5.13.4",
         "@octokit/rest": "^18.0.0",
         "@sindresorhus/slugify": "^1.1.0",
@@ -81,7 +81,7 @@
         "multiparty": "^4.2.1",
         "netlify": "^12.0.0",
         "netlify-headers-parser": "^6.0.2",
-        "netlify-onegraph-internal": "0.8.4",
+        "netlify-onegraph-internal": "0.8.6",
         "netlify-redirect-parser": "^13.0.5",
         "netlify-redirector": "^0.2.1",
         "node-fetch": "^2.6.0",
@@ -1189,13 +1189,13 @@
       "integrity": "sha512-4wMPu9iN3/HL97QblBsBay3E1etIciR84izI3U+4iALY+JHCrI+a2jO0qbAZ/nxKoegypYEaiiqWXylm+/zfrw=="
     },
     "node_modules/@netlify/build": {
-      "version": "27.11.3",
-      "resolved": "https://registry.npmjs.org/@netlify/build/-/build-27.11.3.tgz",
-      "integrity": "sha512-BLMhPSYEkM2YI//KzOH9J9zC6fmss1EHcSWirtdYe6o7Hv8nx2erCqTgLgo887JcZcKaN7HsWbJdyW+jbug5hg==",
+      "version": "27.11.4",
+      "resolved": "https://registry.npmjs.org/@netlify/build/-/build-27.11.4.tgz",
+      "integrity": "sha512-zUbvdN21VGQwg/jVmdpk0qgysUUtBf25hCBxrATt55g3EJVE0ygEm/wX2HV9JjO8OqpydmaaaDdRNmD29aNMAw==",
       "dependencies": {
         "@bugsnag/js": "^7.0.0",
         "@netlify/cache-utils": "^4.0.0",
-        "@netlify/config": "^18.1.3",
+        "@netlify/config": "^18.1.4",
         "@netlify/edge-bundler": "^1.12.1",
         "@netlify/functions-utils": "^4.2.3",
         "@netlify/git-utils": "^4.0.0",
@@ -2011,9 +2011,9 @@
       }
     },
     "node_modules/@netlify/config": {
-      "version": "18.1.3",
-      "resolved": "https://registry.npmjs.org/@netlify/config/-/config-18.1.3.tgz",
-      "integrity": "sha512-StYyw6pAUHMHcnwfPJZrSjz2H2gmpmRtc7hwBCvNzkV9NAcAO7V60cT3oXE7LrITlr5zJaHQLMvUDmvqADJFfA==",
+      "version": "18.1.4",
+      "resolved": "https://registry.npmjs.org/@netlify/config/-/config-18.1.4.tgz",
+      "integrity": "sha512-tM8qDVwBMTYB0n7R6EA6BtuVw9Iq6lNrqE8RjAY4eZ9cjbZkow4M2e0XhmYaNRderLhjYrDlqQr/ttUQA1tZRw==",
       "dependencies": {
         "chalk": "^5.0.0",
         "cron-parser": "^4.1.0",
@@ -3592,9 +3592,9 @@
       "integrity": "sha512-ni6R1xdR8EtH0iB8ixGt9ocuboW+Q8eN4ilTX8lfNHS6Y6Q2S+O/aB2n1BnAgv39wopeQsQ2meL9vfEePURl7w=="
     },
     "node_modules/@netlify/plugins-list": {
-      "version": "6.38.0",
-      "resolved": "https://registry.npmjs.org/@netlify/plugins-list/-/plugins-list-6.38.0.tgz",
-      "integrity": "sha512-fa6BfTCFF3gU2ygS4i175P2Zez8FKXgxKWtjdJPm8eh4UhJ0flIHdooTLZWTpBDL5YiDizk2cviPvOL8fQS93w==",
+      "version": "6.39.0",
+      "resolved": "https://registry.npmjs.org/@netlify/plugins-list/-/plugins-list-6.39.0.tgz",
+      "integrity": "sha512-f8sMN7XFlQQ63km5vhg5y5+E3DXoWEoAe94YOMGlmzifZCCzPS/bEUVaQ07n+iDspy3dOIN97FHAAu0B2Jpe2Q==",
       "engines": {
         "node": "^12.20.0 || ^14.14.0 || >=16.0.0"
       }
@@ -15996,9 +15996,9 @@
       }
     },
     "node_modules/netlify-onegraph-internal": {
-      "version": "0.8.4",
-      "resolved": "https://registry.npmjs.org/netlify-onegraph-internal/-/netlify-onegraph-internal-0.8.4.tgz",
-      "integrity": "sha512-2/yTjAY4qkoKK2aJhR/IX9+V9R7O4RggypAFR7tFBK67uaXDOY1TkImRnTB/9L8yDueyuLlkm0WmNbg/Vq0DfQ==",
+      "version": "0.8.6",
+      "resolved": "https://registry.npmjs.org/netlify-onegraph-internal/-/netlify-onegraph-internal-0.8.6.tgz",
+      "integrity": "sha512-B8xP8eonSNDe9ytJwQVND0eJYDd2JDwedwknJqtiANe11Hw4DHlw/eWZ7CdaTLiRXjD0WbMlW088NZrUZY2rtw==",
       "dependencies": {
         "graphql": "16.5.0",
         "node-fetch": "^2.6.0",
@@ -23384,13 +23384,13 @@
       "integrity": "sha512-4wMPu9iN3/HL97QblBsBay3E1etIciR84izI3U+4iALY+JHCrI+a2jO0qbAZ/nxKoegypYEaiiqWXylm+/zfrw=="
     },
     "@netlify/build": {
-      "version": "27.11.3",
-      "resolved": "https://registry.npmjs.org/@netlify/build/-/build-27.11.3.tgz",
-      "integrity": "sha512-BLMhPSYEkM2YI//KzOH9J9zC6fmss1EHcSWirtdYe6o7Hv8nx2erCqTgLgo887JcZcKaN7HsWbJdyW+jbug5hg==",
+      "version": "27.11.4",
+      "resolved": "https://registry.npmjs.org/@netlify/build/-/build-27.11.4.tgz",
+      "integrity": "sha512-zUbvdN21VGQwg/jVmdpk0qgysUUtBf25hCBxrATt55g3EJVE0ygEm/wX2HV9JjO8OqpydmaaaDdRNmD29aNMAw==",
       "requires": {
         "@bugsnag/js": "^7.0.0",
         "@netlify/cache-utils": "^4.0.0",
-        "@netlify/config": "^18.1.3",
+        "@netlify/config": "^18.1.4",
         "@netlify/edge-bundler": "^1.12.1",
         "@netlify/functions-utils": "^4.2.3",
         "@netlify/git-utils": "^4.0.0",
@@ -23895,9 +23895,9 @@
       }
     },
     "@netlify/config": {
-      "version": "18.1.3",
-      "resolved": "https://registry.npmjs.org/@netlify/config/-/config-18.1.3.tgz",
-      "integrity": "sha512-StYyw6pAUHMHcnwfPJZrSjz2H2gmpmRtc7hwBCvNzkV9NAcAO7V60cT3oXE7LrITlr5zJaHQLMvUDmvqADJFfA==",
+      "version": "18.1.4",
+      "resolved": "https://registry.npmjs.org/@netlify/config/-/config-18.1.4.tgz",
+      "integrity": "sha512-tM8qDVwBMTYB0n7R6EA6BtuVw9Iq6lNrqE8RjAY4eZ9cjbZkow4M2e0XhmYaNRderLhjYrDlqQr/ttUQA1tZRw==",
       "requires": {
         "chalk": "^5.0.0",
         "cron-parser": "^4.1.0",
@@ -24775,9 +24775,9 @@
       "integrity": "sha512-ni6R1xdR8EtH0iB8ixGt9ocuboW+Q8eN4ilTX8lfNHS6Y6Q2S+O/aB2n1BnAgv39wopeQsQ2meL9vfEePURl7w=="
     },
     "@netlify/plugins-list": {
-      "version": "6.38.0",
-      "resolved": "https://registry.npmjs.org/@netlify/plugins-list/-/plugins-list-6.38.0.tgz",
-      "integrity": "sha512-fa6BfTCFF3gU2ygS4i175P2Zez8FKXgxKWtjdJPm8eh4UhJ0flIHdooTLZWTpBDL5YiDizk2cviPvOL8fQS93w=="
+      "version": "6.39.0",
+      "resolved": "https://registry.npmjs.org/@netlify/plugins-list/-/plugins-list-6.39.0.tgz",
+      "integrity": "sha512-f8sMN7XFlQQ63km5vhg5y5+E3DXoWEoAe94YOMGlmzifZCCzPS/bEUVaQ07n+iDspy3dOIN97FHAAu0B2Jpe2Q=="
     },
     "@netlify/run-utils": {
       "version": "4.0.1",
@@ -34229,9 +34229,9 @@
       }
     },
     "netlify-onegraph-internal": {
-      "version": "0.8.4",
-      "resolved": "https://registry.npmjs.org/netlify-onegraph-internal/-/netlify-onegraph-internal-0.8.4.tgz",
-      "integrity": "sha512-2/yTjAY4qkoKK2aJhR/IX9+V9R7O4RggypAFR7tFBK67uaXDOY1TkImRnTB/9L8yDueyuLlkm0WmNbg/Vq0DfQ==",
+      "version": "0.8.6",
+      "resolved": "https://registry.npmjs.org/netlify-onegraph-internal/-/netlify-onegraph-internal-0.8.6.tgz",
+      "integrity": "sha512-B8xP8eonSNDe9ytJwQVND0eJYDd2JDwedwknJqtiANe11Hw4DHlw/eWZ7CdaTLiRXjD0WbMlW088NZrUZY2rtw==",
       "requires": {
         "graphql": "16.5.0",
         "node-fetch": "^2.6.0",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "netlify-cli",
   "description": "Netlify command line tool",
-  "version": "10.17.2",
+  "version": "10.17.5",
   "author": "Netlify Inc.",
   "contributors": [
     "@whitep4nth3r (https://twitter.com/whitep4nth3r)",
@@ -81,6 +81,7 @@
     "Liran Tal <liran.tal@gmail.com> (https://twitter.com/liran_tal)",
     "Louis DeScioli (https://twitter.com/descioli)",
     "Lukas Holzer <lukas.holzer@netlify.com> (https://twitter.com/luka5c0m)",
+    "Lxxyx <Lxxyxzj@gmail.com> (https://twitter.com/Lxxyx_)",
     "Marc Littlemore (https://twitter.com/marclittlemore)",
     "Marcus Weiner",
     "Mark Bello <mark@markbello.dev> (https://markbello.dev)",
@@ -221,12 +222,12 @@
     "prettier": "--ignore-path .gitignore --loglevel=warn \"{src,tools,scripts,site,tests,.github}/**/*.{mjs,cjs,js,md,yml,json,html}\" \"*.{mjs,cjs,js,yml,json,html}\" \".*.{mjs,cjs,js,yml,json,html}\" \"!CHANGELOG.md\" \"!npm-shrinkwrap.json\" \"!**/*/package-lock.json\" \"!.github/**/*.md\""
   },
   "dependencies": {
-    "@netlify/build": "^27.11.3",
-    "@netlify/config": "^18.1.3",
+    "@netlify/build": "^27.11.4",
+    "@netlify/config": "^18.1.4",
     "@netlify/edge-bundler": "^1.12.1",
     "@netlify/framework-info": "^9.2.0",
     "@netlify/local-functions-proxy": "^1.1.1",
-    "@netlify/plugins-list": "^6.38.0",
+    "@netlify/plugins-list": "^6.39.0",
     "@netlify/zip-it-and-ship-it": "^5.13.4",
     "@octokit/rest": "^18.0.0",
     "@sindresorhus/slugify": "^1.1.0",
@@ -292,7 +293,7 @@
     "multiparty": "^4.2.1",
     "netlify": "^12.0.0",
     "netlify-headers-parser": "^6.0.2",
-    "netlify-onegraph-internal": "0.8.4",
+    "netlify-onegraph-internal": "0.8.6",
     "netlify-redirect-parser": "^13.0.5",
     "netlify-redirector": "^0.2.1",
     "node-fetch": "^2.6.0",

package/src/commands/graph/graph-edit.js

@@ -1,6 +1,5 @@
 // @ts-check
 const gitRepoInfo = require('git-repo-info')
-const { OneGraphClient } = require('netlify-onegraph-internal')
 
 const { OneGraphCliClient, ensureCLISession, upsertMergeCLISessionMetadata } = require('../../lib/one-graph/cli-client')
 const {
@@ -40,6 +39,7 @@ const graphEdit = async (options, command) => {
   }
 
   const netlifyToken = await command.authenticate()
+  const { jwt } = await OneGraphCliClient.getGraphJwtForSite({ siteId, nfToken: netlifyToken })
 
   await ensureAppForSite(netlifyToken, siteId)
 
@@ -62,8 +62,7 @@ const graphEdit = async (options, command) => {
       tags: ['netlify-cli', `session:${oneGraphSessionId}`, `git-branch:${branch}`],
     },
     {
-      accessToken: netlifyToken,
-
+      accessToken: jwt,
       siteId,
     },
   )
@@ -76,7 +75,6 @@ const graphEdit = async (options, command) => {
 
   const newMetadata = { docId: persistedDoc.id }
 
-  const { jwt } = await OneGraphClient.getGraphJwtForSite({ siteId, nfToken: netlifyToken })
   await upsertMergeCLISessionMetadata({
     config,
     jwt,

package/src/commands/graph/graph-handler.js

@@ -36,6 +36,8 @@ const graphHandler = async (args, options, command) => {
   const userOperationName = args.operationName
   const userCodegenId = options.codegen
 
+  const handlerOptions = options.data ? JSON.parse(options.data) : {}
+
   let operationName = userOperationName
   if (!operationName) {
     operationName = await autocompleteOperationNames({ netlifyGraphConfig })
@@ -70,7 +72,7 @@ const graphHandler = async (args, options, command) => {
       netlifyGraphConfig,
       schema,
       operationName,
-      handlerOptions: {},
+      handlerOptions,
     })
   } else {
     error(`Failed to parse Netlify GraphQL schema`)
@@ -87,6 +89,7 @@ const createGraphHandlerCommand = (program) =>
     .command('graph:handler')
     .argument('[name]', 'Operation name')
     .option('-c, --codegen <id>', 'The id of the specific code generator to use')
+    .option("-d, --data '<json>'", 'Optional data to pass along to the code generator')
     .description(
       'Generate a handler for a Graph operation given its name. See `graph:operations` for a list of operations.',
     )

package/src/commands/sites/sites-create.js

@@ -1,6 +1,7 @@
 // @ts-check
 
 const slugify = require('@sindresorhus/slugify')
+const { InvalidArgumentError } = require('commander')
 const inquirer = require('inquirer')
 const pick = require('lodash/pick')
 const sample = require('lodash/sample')
@@ -84,7 +85,6 @@ const sitesCreate = async (options, command) => {
     accountSlug = accountSlugInput
   }
 
-  const { name: nameFlag } = options
   let user
   let site
 
@@ -110,7 +110,7 @@ const sitesCreate = async (options, command) => {
       }
     }
   }
-  await inputSiteName(nameFlag)
+  await inputSiteName(options.name)
 
   log()
   log(chalk.greenBright.bold.underline(`Site Created`))
@@ -175,6 +175,16 @@ const sitesCreate = async (options, command) => {
   return site
 }
 
+const MAX_SITE_NAME_LENGTH = 63
+const validateName = function (value) {
+  // netlify sites:create --name <A string of more than 63 words>
+  if (typeof value === 'string' && value.length > MAX_SITE_NAME_LENGTH) {
+    throw new InvalidArgumentError(`--name should be less than 64 characters, input length: ${value.length}`)
+  }
+
+  return value
+}
+
 /**
  * Creates the `netlify sites:create` command
  * @param {import('../base-command').BaseCommand} program
@@ -187,8 +197,8 @@ const createSitesCreateCommand = (program) =>
       `Create an empty site (advanced)
 Create a blank site that isn't associated with any git remote. Will link the site to the current working directory.`,
     )
-    .option('-n, --name [name]', 'name of site')
-    .option('-a, --account-slug [slug]', 'account slug to create the site under')
+    .option('-n, --name <name>', 'name of site', validateName)
+    .option('-a, --account-slug <slug>', 'account slug to create the site under')
     .option('-c, --with-ci', 'initialize CI hooks during site creation')
     .option('-m, --manual', 'force manual CI setup.  Used --with-ci flag')
     .option('--disable-linking', 'create the site without linking it to current directory')

package/src/functions-templates/javascript/scheduled-function/package.json

@@ -15,6 +15,6 @@
   "author": "Netlify",
   "license": "MIT",
   "dependencies": {
-    "@netlify/functions": "^1.0.0"
+    "@netlify/functions": "^1.1.0"
   }
 }

package/src/functions-templates/typescript/hello-world/package-lock.json

@@ -9,15 +9,15 @@
       "version": "1.0.0",
       "license": "MIT",
       "dependencies": {
-        "@netlify/functions": "^1.0.0",
+        "@netlify/functions": "^1.1.0",
         "@types/node": "^14.0.0",
         "typescript": "^4.0.0"
       }
     },
     "node_modules/@netlify/functions": {
-      "version": "1.0.0",
-      "resolved": "https://registry.npmjs.org/@netlify/functions/-/functions-1.0.0.tgz",
-      "integrity": "sha512-7fnJv3vr8uyyyOYPChwoec6MjzsCw1CoRUO2DhQ1BD6bOyJRlD4DUaOOGlMILB2LCT8P24p5LexEGx8AJb7xdA==",
+      "version": "1.1.0",
+      "resolved": "https://registry.npmjs.org/@netlify/functions/-/functions-1.1.0.tgz",
+      "integrity": "sha512-XUFC5nt4iLMrDK+6WjYrDOW9h6XGIQlEk3o++xglFbDKc6dsP+k6rjfz3vl0w8S9Oiosxj3uLaPW18szJc1UgA==",
       "dependencies": {
         "is-promise": "^4.0.0"
       },
@@ -50,9 +50,9 @@
   },
   "dependencies": {
     "@netlify/functions": {
-      "version": "1.0.0",
-      "resolved": "https://registry.npmjs.org/@netlify/functions/-/functions-1.0.0.tgz",
-      "integrity": "sha512-7fnJv3vr8uyyyOYPChwoec6MjzsCw1CoRUO2DhQ1BD6bOyJRlD4DUaOOGlMILB2LCT8P24p5LexEGx8AJb7xdA==",
+      "version": "1.1.0",
+      "resolved": "https://registry.npmjs.org/@netlify/functions/-/functions-1.1.0.tgz",
+      "integrity": "sha512-XUFC5nt4iLMrDK+6WjYrDOW9h6XGIQlEk3o++xglFbDKc6dsP+k6rjfz3vl0w8S9Oiosxj3uLaPW18szJc1UgA==",
       "requires": {
         "is-promise": "^4.0.0"
       }

package/src/functions-templates/typescript/hello-world/package.json

@@ -14,7 +14,7 @@
   "author": "Netlify",
   "license": "MIT",
   "dependencies": {
-    "@netlify/functions": "^1.0.0",
+    "@netlify/functions": "^1.1.0",
     "@types/node": "^14.0.0",
     "typescript": "^4.0.0"
   }

package/src/functions-templates/typescript/scheduled-function/package.json

@@ -15,7 +15,7 @@
   "author": "Netlify",
   "license": "MIT",
   "dependencies": {
-    "@netlify/functions": "^1.0.0",
+    "@netlify/functions": "^1.1.0",
     "@types/node": "^14.18.9",
     "typescript": "^4.5.5"
   }

package/src/lib/one-graph/cli-netlify-graph.js

@@ -685,13 +685,19 @@ const generateHandlerByOperationName = ({
   }
 
   const parsedDoc = parse(currentOperationsDoc)
-  const { functions } = extractFunctionsFromOperationDoc(GraphQL, parsedDoc)
+  const { fragments, functions } = extractFunctionsFromOperationDoc(GraphQL, parsedDoc)
 
-  const operation = Object.values(functions).find(
-    (potentialOperation) => potentialOperation.operationName === operationName,
+  const functionDefinition = Object.values(functions).find(
+    (potentialDefinition) => potentialDefinition.operationName === operationName,
   )
 
-  if (!operation) {
+  const fragmentDefinition = Object.values(fragments).find(
+    (potentialDefinition) => potentialDefinition.fragmentName === operationName,
+  )
+
+  const definition = functionDefinition || fragmentDefinition
+
+  if (!definition) {
     warn(`No operation named ${operationName} was found in the operations doc`)
     return
   }
@@ -701,7 +707,7 @@ const generateHandlerByOperationName = ({
     generate,
     netlifyGraphConfig,
     schema,
-    operationId: operation.id,
+    operationId: definition.id,
     handlerOptions,
   })
 }

package/src/utils/proxy.js

@@ -201,10 +201,9 @@ const serveRedirect = async function ({ match, options, proxy, req, res }) {
     // construct destination URL from redirect rule match
     const dest = new URL(match.to, `${reqUrl.protocol}//${reqUrl.host}`)
 
-    // We pass through request params in one of the following cases:
-    // 1. The redirect rule doesn't have any query params
-    // 2. This is a function redirect https://github.com/netlify/cli/issues/1605
-    if ([...dest.searchParams].length === 0 || isFunction(options.functionsPort, stripOrigin(dest))) {
+    // We pass through request params if the redirect rule
+    // doesn't have any query params
+    if ([...dest.searchParams].length === 0) {
       dest.searchParams.forEach((_, key) => {
         dest.searchParams.delete(key)
       })