fix(cli): keep generate controller no-actions path from writing an index view

Review finding on #3131: the unconditional `actions = result.actions` in
Module.cfc adopted CodeGen's defaulted ["index"] list when the user passed no
actions, so `wheels generate controller Users` started writing
app/views/users/index.cfm — contradicting the documented "passing no actions
creates an empty controller with no view files" behavior retained in the same
code-generation.mdx paragraph.

Surface the caller-requested (post-normalization, pre-default) action list as
result.actions instead: empty input stays empty, so the view loop writes
nothing, while comma-joined tokens still expand to one view per real action.
The controller body keeps its default index() stub as before. Covered by a new
CodeGenSpec case asserting result.actions is empty (and the stub present) for
the no-actions path.

Signed-off-by: Peter Amiri <peter@alurium.com>

Author: bpamiri

cli/lucli/Module.cfc

@@ -3190,9 +3190,11 @@ component extends="modules.BaseModule" {
 			return "";
 		}
 
-		// Use the normalized action list the generator actually wrote (comma-joined
-		// tokens like "index,show" are split into discrete actions) so view files
-		// match the controller methods instead of being named "index,show.cfm" (#3112).
+		// Use the normalized action list from the generator (comma-joined tokens like
+		// "index,show" are split into discrete actions) so view files match the
+		// controller methods instead of being named "index,show.cfm" (#3112). When no
+		// actions were passed result.actions is empty — documented behavior is an
+		// empty controller with no view files, so the view loop below writes nothing.
 		actions = result.actions;
 
 		// Create view files for non-mutation actions

cli/lucli/services/CodeGen.cfc

@@ -149,6 +149,12 @@ component {
 		// each token on commas, trim, and de-duplicate so both forms behave identically.
 		arguments.actions = normalizeActions(arguments.actions);
 
+		// Capture the caller-requested list BEFORE the defaulting below. result.actions
+		// drives the caller's view loop, and the documented contract is "passing no
+		// actions creates an empty controller with no view files" — the index/CRUD
+		// defaults applied next shape the controller body only, never the view files.
+		var requestedActions = arguments.actions;
+
 		// Default actions based on type
 		if (arrayLen(arguments.actions) == 0) {
 			arguments.actions = arguments.crud ? crudActions : ["index"];
@@ -184,10 +190,12 @@ component {
 			context = context
 		);
 
-		// Surface the normalized action list so callers (e.g. Module.cfc's view loop)
-		// render one view file per real action instead of one named after the raw
-		// comma-joined token (#3112).
-		result.actions = arguments.actions;
+		// Surface the normalized caller-requested action list so callers (e.g.
+		// Module.cfc's view loop) render one view file per real action instead of one
+		// named after the raw comma-joined token (#3112). Deliberately the pre-default
+		// list: when no actions were passed this stays empty, so callers write no view
+		// files even though the controller body gets a default index() stub.
+		result.actions = requestedActions;
 		return result;
 	}
 

cli/lucli/tests/specs/services/CodeGenSpec.cfc

@@ -250,6 +250,21 @@ component extends="wheels.wheelstest.system.BaseSpec" {
 					expect(result.actions).toBe(["index", "show"]);
 				});
 
+				it("returns an empty action list when no actions are passed so callers write no views", () => {
+					var result = codegen.generateController(
+						name = "Stubs",
+						actions = [],
+						force = true
+					);
+					var content = fileRead(tempRoot & "/app/controllers/Stubs.cfc");
+					// The controller body still gets the default index() stub...
+					expect(content).toInclude("function index()");
+					// ...but result.actions stays empty so the caller writes no view
+					// files, preserving the documented "no actions => empty controller
+					// with no view files" behavior (PR ##3131 review).
+					expect(result.actions).toBeEmpty();
+				});
+
 			});
 
 			describe("validateName()", () => {