fix: Update commands.run tests for AJV body validation

Body validation is now handled by AJV schema instead of manual checks,
so error messages change from custom strings to AJV validation messages.
Also make triggerId optional in the schema to match actual usage.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: ggazzo

apps/meteor/app/api/server/v1/commands.ts

@@ -240,16 +240,16 @@ API.v1.addRoute(
 	},
 );
 
-const isCommandsRunProps = ajv.compile<{ command: string; params?: string; roomId: string; tmid?: string; triggerId: string }>({
+const isCommandsRunProps = ajv.compile<{ command: string; params?: string; roomId: string; tmid?: string; triggerId?: string }>({
 	type: 'object',
 	properties: {
 		command: { type: 'string' },
 		params: { type: 'string', nullable: true },
 		roomId: { type: 'string' },
 		tmid: { type: 'string', nullable: true },
-		triggerId: { type: 'string' },
+		triggerId: { type: 'string', nullable: true },
 	},
-	required: ['command', 'roomId', 'triggerId'],
+	required: ['command', 'roomId'],
 	additionalProperties: false,
 });
 

apps/meteor/tests/end-to-end/api/commands.ts

@@ -124,7 +124,7 @@ describe('[Commands]', () => {
 				.expect(400)
 				.expect((res) => {
 					expect(res.body).to.have.property('success', false);
-					expect(res.body.error).to.be.equal('You must provide a command to run.');
+					expect(res.body.error).to.be.equal("must have required property 'command'");
 				})
 				.end(done);
 		});
@@ -139,7 +139,7 @@ describe('[Commands]', () => {
 				.expect(400)
 				.expect((res) => {
 					expect(res.body).to.have.property('success', false);
-					expect(res.body.error).to.be.equal('The parameters for the command must be a single string.');
+					expect(res.body.error).to.include('must be string');
 				})
 				.end(done);
 		});
@@ -154,7 +154,7 @@ describe('[Commands]', () => {
 				.expect(400)
 				.expect((res) => {
 					expect(res.body).to.have.property('success', false);
-					expect(res.body.error).to.be.equal("The room's id where to execute this command must be provided and be a string.");
+					expect(res.body.error).to.be.equal("must have required property 'roomId'");
 				})
 				.end(done);
 		});
@@ -171,7 +171,7 @@ describe('[Commands]', () => {
 				.expect(400)
 				.expect((res) => {
 					expect(res.body).to.have.property('success', false);
-					expect(res.body.error).to.be.equal('The tmid parameter when provided must be a string.');
+					expect(res.body.error).to.include('must be string');
 				})
 				.end(done);
 		});

apps/meteor/tests/end-to-end/apps/slash-command-test-simple.ts

@@ -27,7 +27,7 @@ import { IS_EE } from '../../e2e/config/constants';
 				.expect(400)
 				.expect((res) => {
 					expect(res.body).to.have.a.property('success', false);
-					expect(res.body.error).to.be.equal('You must provide a command to run.');
+					expect(res.body.error).to.include('must be string');
 				})
 				.end(done);
 		});