fix(debug): remove incorrect NO_SIDE_EFFECTS annotations

Debug functions (debug, debugDir, debugLog, and *Ns variants) have
side effects (logging, spinner manipulation) and were incorrectly
marked as pure functions, causing bundlers to tree-shake them into
no-ops.

Author: jdalton

CHANGELOG.md

@@ -5,6 +5,15 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.3.6](https://github.com/SocketDev/socket-lib/releases/tag/v1.3.6) - 2025-10-26
+
+### Fixed
+
+- Fixed `debug` module functions being incorrectly tree-shaken as no-ops in bundled output
+  - Removed incorrect `/*@__NO_SIDE_EFFECTS__*/` annotations from `debug()`, `debugDir()`, `debugLog()`, and their `*Ns` variants
+  - These functions have side effects (logging output, spinner manipulation) and should not be removed by bundlers
+  - Fixes issue where `debugLog()` and `debugDir()` were compiled to empty no-op functions
+
 ## [1.3.5](https://github.com/SocketDev/socket-lib/releases/tag/v1.3.5) - 2025-10-26
 
 ### Added

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@socketsecurity/lib",
-  "version": "1.3.5",
+  "version": "1.3.6",
   "license": "MIT",
   "description": "Core utilities and infrastructure for Socket.dev security tools",
   "keywords": [

src/debug.ts

@@ -197,7 +197,6 @@ function isEnabled(namespaces: string | undefined) {
 /**
  * Debug output for object inspection with caller info.
  */
-/*@__NO_SIDE_EFFECTS__*/
 function debugDirNs(
   namespacesOrOpts: NamespacesOrOptions,
   obj: unknown,
@@ -245,7 +244,6 @@ let pointingTriangle: string | undefined
 /**
  * Debug output with caller info.
  */
-/*@__NO_SIDE_EFFECTS__*/
 function debugNs(namespacesOrOpts: NamespacesOrOptions, ...args: unknown[]) {
   const options = extractOptions(namespacesOrOpts)
   const { namespaces } = options
@@ -281,7 +279,6 @@ function debugNs(namespacesOrOpts: NamespacesOrOptions, ...args: unknown[]) {
 /**
  * Debug logging function with caller info.
  */
-/*@__NO_SIDE_EFFECTS__*/
 function debugLogNs(namespacesOrOpts: NamespacesOrOptions, ...args: unknown[]) {
   const options = extractOptions(namespacesOrOpts)
   const { namespaces } = options
@@ -323,7 +320,6 @@ function debugLogNs(namespacesOrOpts: NamespacesOrOptions, ...args: unknown[]) {
  * Second argument is the cache key or message.
  * Optional third argument is metadata object.
  */
-/*@__NO_SIDE_EFFECTS__*/
 function debugCacheNs(
   namespacesOrOpts: NamespacesOrOptions,
   operation: string,
@@ -391,15 +387,13 @@ function isDebugNs(namespaces: string | undefined): boolean {
 /**
  * Debug output with caller info (wrapper for debugNs with default namespace).
  */
-/*@__NO_SIDE_EFFECTS__*/
 function debug(...args: unknown[]): void {
   debugNs('*', ...args)
 }
 
 /**
  * Debug output for object inspection (wrapper for debugDirNs with default namespace).
  */
-/*@__NO_SIDE_EFFECTS__*/
 function debugDir(
   obj: unknown,
   inspectOpts?: InspectOptions | undefined,
@@ -410,7 +404,6 @@ function debugDir(
 /**
  * Debug logging function (wrapper for debugLogNs with default namespace).
  */
-/*@__NO_SIDE_EFFECTS__*/
 function debugLog(...args: unknown[]): void {
   debugLogNs('*', ...args)
 }