refactor(cli-test): rename env add/remove to env set/unset

Aligns with Slack CLI rename of `env add` → `env set` and
`env remove` → `env unset` from slackapi/slack-cli#460.

- Rename `SLACK_TRACE_ENV_ADD_SUCCESS` → `SLACK_TRACE_ENV_SET_SUCCESS`
- Rename `SLACK_TRACE_ENV_REMOVE_SUCCESS` → `SLACK_TRACE_ENV_UNSET_SUCCESS`
- Rename exported `add`/`remove` functions to `set`/`unset`
- Update tests and regenerate docs

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

Author: zimeg

docs/english/reference/cli-test/variables/SlackCLI.md

@@ -328,53 +328,53 @@ command output
 env: object;
 ```
 
-#### env.add()
+#### env.list()
 
 ```ts
-add: (args) => Promise<string>;
+list: (args) => Promise<string>;
 ```
 
-`slack env add`
+`slack env list`
 
 ##### Parameters
 
 ###### args
 
-`ProjectCommandArguments` & `EnvCommandArguments`
+`ProjectCommandArguments`
 
 ##### Returns
 
 `Promise`\<`string`\>
 
 command output
 
-#### env.list()
+#### env.set()
 
 ```ts
-list: (args) => Promise<string>;
+set: (args) => Promise<string>;
 ```
 
-`slack env list`
+`slack env set`
 
 ##### Parameters
 
 ###### args
 
-`ProjectCommandArguments`
+`ProjectCommandArguments` & `EnvCommandArguments`
 
 ##### Returns
 
 `Promise`\<`string`\>
 
 command output
 
-#### env.remove()
+#### env.unset()
 
 ```ts
-remove: (args) => Promise<string>;
+unset: (args) => Promise<string>;
 ```
 
-`slack env remove`
+`slack env unset`
 
 ##### Parameters
 

docs/english/reference/cli-test/variables/SlackTracerId.md

@@ -192,12 +192,6 @@ SLACK_TRACE_DATASTORE_COUNT_SUCCESS: string = 'SLACK_TRACE_DATASTORE_COUNT_SUCCE
 SLACK_TRACE_DATASTORE_COUNT_TOTAL: string = 'SLACK_TRACE_DATASTORE_COUNT_TOTAL';
 ```
 
-### SLACK\_TRACE\_ENV\_ADD\_SUCCESS
-
-```ts
-SLACK_TRACE_ENV_ADD_SUCCESS: string = 'SLACK_TRACE_ENV_ADD_SUCCESS';
-```
-
 ### SLACK\_TRACE\_ENV\_LIST\_COUNT
 
 ```ts
@@ -210,10 +204,16 @@ SLACK_TRACE_ENV_LIST_COUNT: string = 'SLACK_TRACE_ENV_LIST_COUNT';
 SLACK_TRACE_ENV_LIST_VARIABLES: string = 'SLACK_TRACE_ENV_LIST_VARIABLES';
 ```
 
-### SLACK\_TRACE\_ENV\_REMOVE\_SUCCESS
+### SLACK\_TRACE\_ENV\_SET\_SUCCESS
+
+```ts
+SLACK_TRACE_ENV_SET_SUCCESS: string = 'SLACK_TRACE_ENV_SET_SUCCESS';
+```
+
+### SLACK\_TRACE\_ENV\_UNSET\_SUCCESS
 
 ```ts
-SLACK_TRACE_ENV_REMOVE_SUCCESS: string = 'SLACK_TRACE_ENV_REMOVE_SUCCESS';
+SLACK_TRACE_ENV_UNSET_SUCCESS: string = 'SLACK_TRACE_ENV_UNSET_SUCCESS';
 ```
 
 ### SLACK\_TRACE\_FEEDBACK\_MESSAGE

packages/cli-test/src/cli/commands/env.test.ts

@@ -24,13 +24,13 @@ describe('env commands', () => {
     sandbox.restore();
   });
 
-  describe('add method', () => {
-    it('should invoke `env add <key> <value>`', async () => {
-      await env.add({ appPath: '/some/path', secretKey: 'key', secretValue: 'value' });
+  describe('set method', () => {
+    it('should invoke `env set <key> <value>`', async () => {
+      await env.set({ appPath: '/some/path', secretKey: 'key', secretValue: 'value' });
       sandbox.assert.calledWith(
         spawnSpy,
         sinon.match.string,
-        sinon.match.array.contains(['env', 'add', 'key', 'value']),
+        sinon.match.array.contains(['env', 'set', 'key', 'value']),
       );
     });
   });
@@ -40,10 +40,10 @@ describe('env commands', () => {
       sandbox.assert.calledWith(spawnSpy, sinon.match.string, sinon.match.array.contains(['env', 'list']));
     });
   });
-  describe('remove method', () => {
-    it('should invoke `env remove <key>`', async () => {
-      await env.remove({ appPath: '/some/path', secretKey: 'key' });
-      sandbox.assert.calledWith(spawnSpy, sinon.match.string, sinon.match.array.contains(['env', 'remove', 'key']));
+  describe('unset method', () => {
+    it('should invoke `env unset <key>`', async () => {
+      await env.unset({ appPath: '/some/path', secretKey: 'key' });
+      sandbox.assert.calledWith(spawnSpy, sinon.match.string, sinon.match.array.contains(['env', 'unset', 'key']));
     });
   });
 });

packages/cli-test/src/cli/commands/env.ts

@@ -9,11 +9,11 @@ export interface EnvCommandArguments {
 }
 
 /**
- * `slack env add`
+ * `slack env set`
  * @returns command output
  */
-export const add = async function envAdd(args: ProjectCommandArguments & EnvCommandArguments): Promise<string> {
-  const cmd = new SlackCLIProcess(['env', 'add', args.secretKey, args.secretValue], args);
+export const set = async function envSet(args: ProjectCommandArguments & EnvCommandArguments): Promise<string> {
+  const cmd = new SlackCLIProcess(['env', 'set', args.secretKey, args.secretValue], args);
   const proc = await cmd.execAsync({
     cwd: args.appPath,
   });
@@ -33,21 +33,21 @@ export const list = async function envList(args: ProjectCommandArguments): Promi
 };
 
 /**
- * `slack env remove`
+ * `slack env unset`
  * @returns command output
  */
-export const remove = async function envRemove(
+export const unset = async function envUnset(
   args: ProjectCommandArguments & Pick<EnvCommandArguments, 'secretKey'>,
 ): Promise<string> {
-  const cmd = new SlackCLIProcess(['env', 'remove', args.secretKey], args);
+  const cmd = new SlackCLIProcess(['env', 'unset', args.secretKey], args);
   const proc = await cmd.execAsync({
     cwd: args.appPath,
   });
   return proc.output;
 };
 
 export default {
-  add,
+  set,
   list,
-  remove,
+  unset,
 };

packages/cli-test/src/utils/constants.ts

@@ -33,10 +33,10 @@ export const SlackTracerId = {
   SLACK_TRACE_DATASTORE_COUNT_DATASTORE: 'SLACK_TRACE_DATASTORE_COUNT_DATASTORE',
   SLACK_TRACE_DATASTORE_COUNT_SUCCESS: 'SLACK_TRACE_DATASTORE_COUNT_SUCCESS',
   SLACK_TRACE_DATASTORE_COUNT_TOTAL: 'SLACK_TRACE_DATASTORE_COUNT_TOTAL',
-  SLACK_TRACE_ENV_ADD_SUCCESS: 'SLACK_TRACE_ENV_ADD_SUCCESS',
+  SLACK_TRACE_ENV_SET_SUCCESS: 'SLACK_TRACE_ENV_SET_SUCCESS',
   SLACK_TRACE_ENV_LIST_COUNT: 'SLACK_TRACE_ENV_LIST_COUNT',
   SLACK_TRACE_ENV_LIST_VARIABLES: 'SLACK_TRACE_ENV_LIST_VARIABLES',
-  SLACK_TRACE_ENV_REMOVE_SUCCESS: 'SLACK_TRACE_ENV_REMOVE_SUCCESS',
+  SLACK_TRACE_ENV_UNSET_SUCCESS: 'SLACK_TRACE_ENV_UNSET_SUCCESS',
   SLACK_TRACE_FEEDBACK_MESSAGE: 'SLACK_TRACE_FEEDBACK_MESSAGE',
   SLACK_TRACE_MANIFEST_VALIDATE_SUCCESS: 'SLACK_TRACE_MANIFEST_VALIDATE_SUCCESS',
   SLACK_TRACE_PLATFORM_DEPLOY_SUCCESS: 'SLACK_TRACE_PLATFORM_DEPLOY_SUCCESS',