chore: kebab-case post-processing flag

docs/commands/env.md

@@ -85,7 +85,7 @@ netlify env:get
 **Flags**
 
 - `context` (*string*) - Specify a deploy context or branch (contexts: "production", "deploy-preview", "branch-deploy", "dev")
-- `scope` (*builds | functions | post_processing | runtime | any*) - Specify a scope
+- `scope` (*builds | functions | post-processing | runtime | any*) - Specify a scope
 - `debug` (*boolean*) - Print debugging information
 - `httpProxy` (*string*) - Proxy server address to route requests through.
 - `httpProxyCertificateFilename` (*string*) - Certificate file to use when connecting using a proxy server
@@ -135,7 +135,7 @@ netlify env:list
 **Flags**
 
 - `context` (*string*) - Specify a deploy context or branch (contexts: "production", "deploy-preview", "branch-deploy", "dev")
-- `scope` (*builds | functions | post_processing | runtime | any*) - Specify a scope
+- `scope` (*builds | functions | post-processing | runtime | any*) - Specify a scope
 - `debug` (*boolean*) - Print debugging information
 - `httpProxy` (*string*) - Proxy server address to route requests through.
 - `httpProxyCertificateFilename` (*string*) - Certificate file to use when connecting using a proxy server
@@ -168,7 +168,7 @@ netlify env:set
 **Flags**
 
 - `context` (*string*) - Specify a deploy context or branch (contexts: "production", "deploy-preview", "branch-deploy", "dev") (default: all contexts)
-- `scope` (*builds | functions | post_processing | runtime*) - Specify a scope (default: all scopes)
+- `scope` (*builds | functions | post-processing | runtime*) - Specify a scope (default: all scopes)
 - `debug` (*boolean*) - Print debugging information
 - `httpProxy` (*string*) - Proxy server address to route requests through.
 - `httpProxyCertificateFilename` (*string*) - Certificate file to use when connecting using a proxy server

src/commands/env/env-get.cjs

@@ -77,7 +77,7 @@ const createEnvGetCommand = (program) =>
     )
     .addOption(
       new Option('-s, --scope <scope>', 'Specify a scope')
-        .choices(['builds', 'functions', 'post_processing', 'runtime', 'any'])
+        .choices(['builds', 'functions', 'post-processing', 'runtime', 'any'])
         .default('any'),
     )
     .addExamples([

src/commands/env/env-list.cjs

@@ -140,7 +140,7 @@ const createEnvListCommand = (program) =>
     )
     .addOption(
       new Option('-s, --scope <scope>', 'Specify a scope')
-        .choices(['builds', 'functions', 'post_processing', 'runtime', 'any'])
+        .choices(['builds', 'functions', 'post-processing', 'runtime', 'any'])
         .default('any'),
     )
     .addExamples([

src/commands/env/env-set.cjs

@@ -165,7 +165,7 @@ const createEnvSetCommand = (program) =>
       new Option('-s, --scope <scope...>', 'Specify a scope (default: all scopes)').choices([
         'builds',
         'functions',
-        'post_processing',
+        'post-processing',
         'runtime',
       ]),
     )

src/utils/env/index.cjs

@@ -150,7 +150,7 @@ const getEnvelopeEnv = async ({ api, context = 'dev', env, key = '', scope = 'an
   const configFileEnv = filterEnvBySource(env, 'configFile')
 
   // filter out configFile env vars if a non-configFile scope is passed
-  const includeConfigEnvVars = ['any', 'builds', 'post_processing'].includes(scope)
+  const includeConfigEnvVars = /any|builds|post[-_]processing/.test(scope)
 
   // Sources of environment variables, in ascending order of precedence.
   return {
@@ -168,11 +168,13 @@ const getEnvelopeEnv = async ({ api, context = 'dev', env, key = '', scope = 'an
  * @returns {string} A human-readable, comma-separated list of scopes
  */
 const getHumanReadableScopes = (scopes) => {
-  const HUMAN_SCOPES = {
-    builds: 'Builds',
-    functions: 'Functions',
-    post_processing: 'Post processing',
-    runtime: 'Runtime',
+  const HUMAN_SCOPES = ['Builds', 'Functions', 'Runtime', 'Post processing']
+  const SCOPES_MAP = {
+    builds: HUMAN_SCOPES[0],
+    functions: HUMAN_SCOPES[1],
+    runtime: HUMAN_SCOPES[2],
+    post_processing: HUMAN_SCOPES[3],
+    'post-processing': HUMAN_SCOPES[3],
   }
   if (!scopes) {
     // if `scopes` is not available, the env var comes from netlify.toml
@@ -183,7 +185,7 @@ const getHumanReadableScopes = (scopes) => {
     // shorthand instead of listing every available scope
     return 'All'
   }
-  return scopes.map((scope) => HUMAN_SCOPES[scope]).join(', ')
+  return scopes.map((scope) => SCOPES_MAP[scope]).join(', ')
 }
 
 /**

tests/integration/650.command.envelope.test.cjs

@@ -366,7 +366,7 @@ test('env:set --scope should set the scope of an existing env var without needin
 
     await withMockApi(routes, async ({ apiUrl, requests }) => {
       const cliResponse = await callCli(
-        ['env:set', 'EXISTING_VAR', '--scope', 'runtime', 'post_processing', '--json'],
+        ['env:set', 'EXISTING_VAR', '--scope', 'runtime', 'post-processing', '--json'],
         getCLIOptions({ builder, apiUrl }),
         true,
       )

tests/unit/utils/env/index.test.cjs

@@ -118,8 +118,10 @@ test('should convert scope keys into a human-readable list', (t) => {
   t.is(getHumanReadableScopes([]), '')
   t.is(getHumanReadableScopes(), 'Builds, Post processing')
   t.is(getHumanReadableScopes(['post_processing']), 'Post processing')
+  t.is(getHumanReadableScopes(['post-processing']), 'Post processing')
   t.is(getHumanReadableScopes(['builds', 'functions']), 'Builds, Functions')
   t.is(getHumanReadableScopes(['builds', 'functions', 'runtime', 'post_processing']), 'All')
+  t.is(getHumanReadableScopes(['builds', 'functions', 'runtime', 'post-processing']), 'All')
 })
 
 test('should normalize a branch name or context', (t) => {