Add CHANGELOG.md, change prefix to [debugger], default consoleOutput/logExceptions to true

zspitzer · zspitzer · commit 52298e5b36a1 · 2026-02-12T16:33:15.000+11:00
diff --git a/.gitignore b/.gitignore
@@ -28,6 +28,8 @@ test-output.txt
 # Markdown (except docs)
 *.md
 !README.md
+!CHANGELOG.md
 !AGENTS.md
 !JAVA_AGENT.md
 !docs/*.md
+examples/docker/.vscode/settings.json
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -0,0 +1,59 @@
+# Changelog
+
+## [3.0.0.3-SNAPSHOT] - 2026-02-11
+
+### Fixed
+
+- **Exception breakpoints now work in native mode (Lucee 7.1+)** - Fixed critical bug where onException() was never called for uncaught exceptions. Added onException() call in Lucee's PageContextImpl.java execute() catch block.
+- Docker example .lco file deployment - Fixed COPY path in Dockerfile to correctly reference files from build context
+
+### Changed
+
+- Log prefix changed from `[luceedebug]` to `[debugger]` for consistency
+- Exception logging now defaults to `true` (was `false`) - exception details are lost after continuing, so logging them makes sense
+- Console output streaming now defaults to `true` (was `false`) - it's a debug tool, DX matters more than a bit of overhead
+- onException() calls now log at DEBUG level instead of INFO - reduces noise in debug console
+
+### Added
+
+- Test infrastructure for exception logging configuration (logLevel and logExceptions parameters)
+- Three new tests for exception logging behavior:
+  - testExceptionLoggingWithLogExceptionsEnabled
+  - testExceptionLoggingDisabled
+  - testOnExceptionCalledLogsAtDebugLevel
+
+### Removed
+
+- Diagnostic logging from isDapClientConnected() - simplified to just return the boolean flag
+
+## [3.0.0.2-SNAPSHOT] - 2026-01-30
+
+### Changed
+
+- Reduced client logging verbosity for cleaner debug output
+- Improved Docker example documentation with detailed logging explanations
+
+### Added
+
+- Documentation for `LUCEE_DEBUGGER_DEBUG` environment variable for verbose server-side logging
+- IDE integration tips for Neovim and JetBrains in Docker example README
+
+## [3.0.0.1-SNAPSHOT] - 2026-01-28
+
+### Added
+
+- **Docker example** - Complete working example with Lucee 7.1 in Docker, includes VS Code, Neovim, and JetBrains configurations
+- `LUCEE_DAP_HOST` environment variable for binding DAP server to specific addresses (default: localhost, use `0.0.0.0` for Docker)
+- Maven Central publishing support in CI/CD
+- AGENTS.md documentation for building from source
+
+### Changed
+
+- **Migrated from Gradle to Maven** build system
+- VS Code extension migrated to lucee publisher for Marketplace release
+- Updated tests to use published Lucee 7.1 alpha instead of custom builds
+
+### Fixed
+
+- .lex deployment with correct GAV (Group-Artifact-Version) in manifest
+- CI deploy job GPG configuration
diff --git a/README.md b/README.md
@@ -30,7 +30,7 @@ The simplest and most efficient way to use luceedebug. No JVM configuration requ
 
 **Setup:**
 
-1. Install the extension via Lucee Admin, using `LUCEE_EXTENSIONS=org.lucee:debugger-extension:3.0.0.2-SNAPSHOT`, or deploy the `.lex` file to your extensions folder
+1. Install the extension via Lucee Admin, using `LUCEE_EXTENSIONS=org.lucee:debugger-extension:3.0.0.3-SNAPSHOT`, or deploy the `.lex` file to your extensions folder
 2. Set environment variables:
 
    ```bash
@@ -98,9 +98,9 @@ These options control how the debug adapter behaves. They apply regardless of wh
 | `port` | DAP server port (must match `LUCEE_DAP_PORT`) |
 | `secret` | Authentication secret (must match `LUCEE_DAP_SECRET`, required for extension mode) |
 | `logLevel` | Log verbosity: `error`, `info`, `debug`, `trace` (default: info) |
-| `consoleOutput` | Stream console output to debug console (extension mode only, default: false) |
+| `consoleOutput` | Stream console output to debug console (extension mode only, default: true) |
 | `evaluation` | Enable expression evaluation in console/watch/hover (default: true) |
-| `logExceptions` | Log exceptions to the debug console (default: false) |
+| `logExceptions` | Log exception stacktraces to the debug console (default: true) |
 
 ### DAP Capabilities
 
diff --git a/agent/pom.xml b/agent/pom.xml
@@ -38,7 +38,7 @@
 	</scm>
 
 	<properties>
-		<revision>3.0.0.2-SNAPSHOT</revision>
+		<revision>3.0.0.3-SNAPSHOT</revision>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
 		<maven.compiler.source>11</maven.compiler.source>
 		<maven.compiler.target>11</maven.compiler.target>
diff --git a/examples/docker/Dockerfile b/examples/docker/Dockerfile
@@ -1,2 +1,3 @@
-FROM lucee/lucee:7.1.0.46-SNAPSHOT-nginx-tomcat11.0-jdk25-temurin-noble
+FROM lucee/lucee:7.1.0.48-SNAPSHOT-nginx-tomcat11.0-jdk25-temurin-noble
 # COPY target/debugger-extension-*.lex /opt/lucee/server/lucee-server/deploy/
+# COPY examples/docker/*.lco /opt/lucee/server/lucee-server/deploy/
diff --git a/examples/docker/README.md b/examples/docker/README.md
@@ -86,7 +86,7 @@ See the [main README](../../README.md) for all configuration options, DAP capabi
 
 The Lucee 7.1+ Docker image is used, with the debugger extension being installed via an env var:
 
-`LUCEE_EXTENSIONS: org.lucee:debugger-extension:3.0.0.2-SNAPSHOT`.
+`LUCEE_EXTENSIONS: org.lucee:debugger-extension:3.0.0.3-SNAPSHOT`.
 
 Three env vars enable the debugger:
 
diff --git a/examples/docker/app/index.cfm b/examples/docker/app/index.cfm
@@ -1,9 +1,29 @@
 <cfscript>
+
+	deploy = directoryList("/opt/lucee/server/lucee-server/deploy/");
+	dump(deploy);
+	flush;
+
 	systemOutput("Request: " & cgi.request_url, true);
 	greeting = "Hello from Lucee!";
 	now = now();
+	
+
+	systemOutput("Triggering breakpoint via BIF", true);
+	breakpoint();
 </cfscript>
 <cfoutput>
 	<h1>#greeting#</h1>
 	<p>#dateTimeFormat( now, "full" )#</p>
+	
 </cfoutput>
+<cfscript>
+
+	systemOutput("Triggering breakpoint via throw", true);
+
+	function throwWithStack(){
+		throw("zac");
+	}
+
+	throwWithStack();
+</cfscript>
diff --git a/examples/docker/docker-compose.yml b/examples/docker/docker-compose.yml
@@ -7,10 +7,10 @@ services:
       LUCEE_DAP_SECRET: my-secret
       LUCEE_DAP_PORT: 10000
       LUCEE_DAP_HOST: 0.0.0.0
-      LUCEE_LOGGING_FORCE_APPENDER: console
+      LUCEE_LOGGING_FORCE_APPENDER_: console
       LUCEE_LOGGING_FORCE_LEVEL: info
       LUCEE_DEBUGGER_DEBUG: false
-      LUCEE_EXTENSIONS: org.lucee:debugger-extension:3.0.0.2-SNAPSHOT
+      LUCEE_EXTENSIONS: org.lucee:debugger-extension:3.0.0.3-SNAPSHOT
     ports:
       - "8888:8888"
       - "10000:10000"
diff --git a/pom.xml b/pom.xml
@@ -4,7 +4,7 @@
 	<modelVersion>4.0.0</modelVersion>
 	<groupId>org.lucee</groupId>
 	<artifactId>debugger-extension</artifactId>
-	<version>3.0.0.2-SNAPSHOT</version>
+	<version>3.0.0.3-SNAPSHOT</version>
 	<packaging>pom</packaging>
 	<name>Debugger Extension</name>
 
diff --git a/source/java/src/org/lucee/extension/debugger/DapServer.java b/source/java/src/org/lucee/extension/debugger/DapServer.java
@@ -583,13 +583,13 @@ private void configureLogging(Map<String, Object> args) {
             }
         }
 
-        // logExceptions - default false
-        Log.setLogExceptions(toBooleanValue(args.get("logExceptions"), false));
+        // logExceptions - default true (exception details are lost after continuing)
+        Log.setLogExceptions(toBooleanValue(args.get("logExceptions"), true));
 
-        // consoleOutput - default false (streams System.out/err to debug console)
+        // consoleOutput - default true (streams System.out/err to debug console)
         // Only available in native mode
         if (luceeVm_ instanceof NativeLuceeVm) {
-            NativeDebuggerListener.setConsoleOutput(toBooleanValue(args.get("consoleOutput"), false));
+            NativeDebuggerListener.setConsoleOutput(toBooleanValue(args.get("consoleOutput"), true));
         }
     }
 
diff --git a/source/java/src/org/lucee/extension/debugger/EnvUtil.java b/source/java/src/org/lucee/extension/debugger/EnvUtil.java
@@ -13,7 +13,7 @@ private EnvUtil() {}
 	 * System property takes precedence. Env var name is derived from property name
 	 * by uppercasing and replacing dots with underscores.
 	 *
-	 * @param propertyName e.g. "lucee.debugger.enabled"
+	 * @param propertyName e.g. "lucee.dap.secret"
 	 * @return the value, or null if not set
 	 */
 	public static String getSystemPropOrEnvVar(String propertyName) {
diff --git a/source/java/src/org/lucee/extension/debugger/Log.java b/source/java/src/org/lucee/extension/debugger/Log.java
@@ -16,8 +16,8 @@
  * - Writes errors to Lucee's exception.log when available
  */
 public class Log {
-	private static final String PREFIX = "[luceedebug] ";
-	private static final String APP_NAME = "luceedebug";
+	private static final String PREFIX = "[debugger] ";
+	private static final String APP_NAME = "debugger";
 
 	// ANSI escape codes (for console/tomcat output)
 	private static final String ANSI_RESET = "\u001b[0m";
@@ -32,8 +32,8 @@ public class Log {
 	// Runtime settings from launch.json
 	private static volatile boolean colorLogs = true;
 	private static volatile LogLevel logLevel = LogLevel.INFO;
-	private static volatile boolean logExceptions = false;
-	private static volatile boolean consoleOutput = false;
+	private static volatile boolean logExceptions = true;
+	private static volatile boolean consoleOutput = true;
 
 	// Internal debugging - only enabled via env var LUCEE_DEBUGGER_DEBUG
 	private static final boolean internalDebug;
@@ -229,6 +229,7 @@ public static void exception(Throwable t) {
 
 		// Get full CFML stack trace
 		String stackTrace = ExceptionUtil.getCfmlStackTraceOrFallback(t);
+		debug("Stack trace extraction result: " + (stackTrace == null ? "null" : stackTrace.isEmpty() ? "empty" : stackTrace.length() + " chars"));
 		if (stackTrace != null && !stackTrace.isEmpty()) {
 			for (String line : stackTrace.split("\n")) {
 				if (!line.isEmpty()) {
@@ -237,6 +238,11 @@ public static void exception(Throwable t) {
 			}
 		} else {
 			sb.append("\n  at unknown");
+			// Also dump Java stack trace for debugging
+			debug("Java stack trace for " + t.getClass().getName() + ":");
+			for (StackTraceElement ste : t.getStackTrace()) {
+				debug("  " + ste.toString());
+			}
 		}
 		sendToDap(sb.toString(), OutputEventArgumentsCategory.STDERR);
 	}
diff --git a/source/java/src/org/lucee/extension/debugger/coreinject/NativeDebuggerListener.java b/source/java/src/org/lucee/extension/debugger/coreinject/NativeDebuggerListener.java
@@ -168,7 +168,7 @@ private static class CachedExecutableLines {
 	 * Flag to forward System.out/err to DAP client.
 	 * Set via launch.json consoleOutput option.
 	 */
-	private static volatile boolean consoleOutput = false;
+	private static volatile boolean consoleOutput = true;
 
 	/**
 	 * Fast-path flag: true when there's anything that could cause a suspend.
diff --git a/test/cfml/DapClient.cfc b/test/cfml/DapClient.cfc
@@ -63,7 +63,7 @@ component {
 		return response;
 	}
 
-	public struct function attach( required string secret, struct pathTransforms = {}, boolean consoleOutput = false ) {
+	public struct function attach( required string secret, struct pathTransforms = {}, boolean consoleOutput = false, string logLevel = "", boolean logExceptions = false ) {
 		var args = {
 			"secret": arguments.secret
 		};
@@ -73,6 +73,12 @@ component {
 		if ( arguments.consoleOutput ) {
 			args[ "consoleOutput" ] = true;
 		}
+		if ( len( arguments.logLevel ) ) {
+			args[ "logLevel" ] = arguments.logLevel;
+		}
+		if ( arguments.logExceptions ) {
+			args[ "logExceptions" ] = true;
+		}
 		var response = sendRequest( "attach", args );
 		// Wait for initialized event from server
 		waitForEvent( "initialized", 5000 );
diff --git a/test/cfml/DapTestCase.cfm b/test/cfml/DapTestCase.cfm
@@ -14,7 +14,7 @@ variables.debuggeeArtifactPath = server.system.environment.DEBUGGEE_ARTIFACT_PAT
 variables.httpThread = "";
 variables.httpResult = {};
 
-function setupDap( boolean attach = true, boolean consoleOutput = false ) {
+function setupDap( boolean attach = true, boolean consoleOutput = false, string logLevel = "", boolean logExceptions = false ) {
 	variables.dapHost = server.system.environment.DAP_HOST ?: "localhost";
 	variables.dapPort = val( server.system.environment.DAP_PORT ?: 10000 );
 	variables.debuggeeHttp = server.system.environment.DEBUGGEE_HTTP ?: "http://localhost:8888";
@@ -54,7 +54,7 @@ function setupDap( boolean attach = true, boolean consoleOutput = false ) {
 	systemOutput( "DapTestCase: Capabilities: #serializeJSON( variables.capabilities )#", true );
 
 	if ( arguments.attach ) {
-		variables.dap.attach( variables.dapSecret, {}, arguments.consoleOutput );
+		variables.dap.attach( variables.dapSecret, {}, arguments.consoleOutput, arguments.logLevel, arguments.logExceptions );
 		variables.dap.configurationDone();
 	}
 }
diff --git a/test/cfml/ExceptionBreakpointsTest.cfc b/test/cfml/ExceptionBreakpointsTest.cfc
@@ -185,4 +185,111 @@ component extends="org.lucee.cfml.test.LuceeTestCase" labels="dap" {
 		expect( result ).toHaveKey( "status" );
 	}
 
+	// ========== Exception Logging ==========
+
+	function testExceptionLoggingWithLogExceptionsEnabled() skip="notSupportsExceptionBreakpoints" {
+		// Reconnect with exception logging enabled
+		teardownDap();
+		setupDap( logLevel = "info", logExceptions = true );
+
+		dap.setExceptionBreakpoints( [ "uncaught" ] );
+
+		// Trigger uncaught exception
+		triggerArtifact( "exception-target.cfm", { throwException: true, catchException = false }, true );
+
+		var stopped = dap.waitForEvent( "stopped", 2000 );
+
+		// Drain output events
+		var events = dap.drainEvents();
+		var foundExceptionLog = false;
+
+		systemOutput( "Checking #events.len()# events for exception logging", true );
+
+		for ( var event in events ) {
+			if ( event.event == "output" ) {
+				systemOutput( "Output event: category=#event.body.category# output=#event.body.output#", true );
+				// Should see exception details in output
+				if ( event.body.output contains "TestException" && event.body.output contains "Intentional test exception" ) {
+					foundExceptionLog = true;
+					expect( event.body.category ).toBe( "stderr", "Exception logging should use stderr category" );
+				}
+			}
+		}
+
+		expect( foundExceptionLog ).toBeTrue( "Exception should be logged to debug console when logExceptions=true" );
+
+		cleanupThread( stopped.body.threadId );
+
+		// Restore original setup for subsequent tests
+		teardownDap();
+		setupDap();
+		variables.targetFile = getArtifactPath( "exception-target.cfm" );
+	}
+
+	function testExceptionLoggingDisabled() skip="notSupportsExceptionBreakpoints" {
+		// Reconnect with exception logging explicitly disabled
+		teardownDap();
+		setupDap( logExceptions = false );
+
+		dap.setExceptionBreakpoints( [ "uncaught" ] );
+
+		// Trigger uncaught exception
+		triggerArtifact( "exception-target.cfm", { throwException: true, catchException: false }, true );
+
+		var stopped = dap.waitForEvent( "stopped", 2000 );
+
+		// Drain output events
+		var events = dap.drainEvents();
+		var foundExceptionLog = false;
+
+		for ( var event in events ) {
+			if ( event.event == "output" && event.body.output contains "TestException" ) {
+				foundExceptionLog = true;
+			}
+		}
+
+		// Exception details should NOT be logged when logExceptions is false
+		expect( foundExceptionLog ).toBeFalse( "Exception should NOT be logged to debug console when logExceptions=false" );
+
+		cleanupThread( stopped.body.threadId );
+	}
+
+	function testOnExceptionCalledLogsAtDebugLevel() skip="notSupportsExceptionBreakpoints" {
+		// Reconnect with debug log level to see onException calls
+		teardownDap();
+		setupDap( logLevel = "debug" );
+
+		dap.setExceptionBreakpoints( [ "uncaught" ] );
+
+		// Trigger uncaught exception
+		triggerArtifact( "exception-target.cfm", { throwException: true, catchException: false }, true );
+
+		var stopped = dap.waitForEvent( "stopped", 2000 );
+
+		// Drain output events
+		var events = dap.drainEvents();
+		var foundOnExceptionLog = false;
+
+		systemOutput( "Checking events for onException logging at debug level", true );
+
+		for ( var event in events ) {
+			if ( event.event == "output" ) {
+				systemOutput( "Output event: #event.body.output#", true );
+				// Should see onException being called at DEBUG level
+				if ( event.body.output contains "onException" ) {
+					foundOnExceptionLog = true;
+				}
+			}
+		}
+
+		expect( foundOnExceptionLog ).toBeTrue( "onException calls should be logged at DEBUG level" );
+
+		cleanupThread( stopped.body.threadId );
+
+		// Restore original setup
+		teardownDap();
+		setupDap();
+		variables.targetFile = getArtifactPath( "exception-target.cfm" );
+	}
+
 }

Original file line number	Diff line number	Diff line change
`@@ -583,13 +583,13 @@ private void configureLogging(Map<String, Object> args) {`
`583`	`583`	`}`
`584`	`584`	`}`
`585`	`585`
`586`		`- // logExceptions - default false`
`587`		`- Log.setLogExceptions(toBooleanValue(args.get("logExceptions"), false));`
	`586`	`+ // logExceptions - default true (exception details are lost after continuing)`
	`587`	`+ Log.setLogExceptions(toBooleanValue(args.get("logExceptions"), true));`
`588`	`588`
`589`		`- // consoleOutput - default false (streams System.out/err to debug console)`
	`589`	`+ // consoleOutput - default true (streams System.out/err to debug console)`
`590`	`590`	`// Only available in native mode`
`591`	`591`	`if (luceeVm_ instanceof NativeLuceeVm) {`
`592`		`- NativeDebuggerListener.setConsoleOutput(toBooleanValue(args.get("consoleOutput"), false));`
	`592`	`+ NativeDebuggerListener.setConsoleOutput(toBooleanValue(args.get("consoleOutput"), true));`
`593`	`593`	`}`
`594`	`594`	`}`
`595`	`595`