amqplib 0.10.8 → 1.0.0

package/.claude/commands/lint-disable.md

@@ -0,0 +1,9 @@
+Please follow the following process when I ask you to disable a lint rule using /lint-disable $rule
+
+1. Create and change to a new branch. Ask for the branch name.
+2. If the rule is off by default, delete it (See https://biomejs.dev/linter/javascript/rules/ for rule details)
+3. If the rule is on by default, configure it to off in biome.json
+4. Update the project changelog
+5. Add the changes
+6. Commit the changes
+7. Push the changes

package/.claude/commands/lint-enable.md

@@ -0,0 +1,12 @@
+Please follow the following process when I ask you to enable a lint rule using /lint-enable $rule
+
+1. Create and change to a new branch. Ask for the branch name.
+2. Configure the rule to error in biome.json
+3. Run lint (npm run lint). If there are NO errors, goto step 8
+4. Use biome to SAFELY auto fix (npm run lint -- --fix). If there are errors stop and ask for instruction.
+5. Run the tests (npm test). If there are errors, stop and ask for instruction.
+6. If the rule defaults to error, delete it (See https://biomejs.dev/linter/javascript/rules/ for rule details)
+7. Update the project changelog
+8. Add the changes
+9. Commit the changes
+10. Push the changes

package/.claude/settings.local.json

@@ -0,0 +1,16 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(npx oxlint:*)",
+      "Bash(npm t)",
+      "Bash(git add:*)",
+      "Bash(git commit:*)",
+      "Bash(npm run lint:*)",
+      "Bash(git checkout:*)",
+      "Bash(npx biome lint:*)",
+      "Bash(node:*)"
+    ],
+    "deny": [],
+    "ask": []
+  }
+}

package/.github/workflows/publish.yml

@@ -15,7 +15,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [10.x, 12.x, 14.x, 16.x, 18.x, 20.x]
+        node-version: [18.x, 20.x, 22.x, 24.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:

package/.github/workflows/test.yml

@@ -20,7 +20,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [10.x, 12.x, 14.x, 16.x, 18.x, 20.x]
+        node-version: [18.x, 20.x, 22.x, 24.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:

package/CHANGELOG.md

@@ -1,5 +1,39 @@
 # Change log for amqplib
 
+## v1.0.0
+- **BREAKING**: Update minimum Node.js requirement to v18
+- Introduce Biome formatter for consistent code formatting
+- Add automated formatting via pre-commit hooks using Lefthook
+- Format entire codebase with standardised formatting rules
+- Add npm format script for manual code formatting
+- Enable noUnusedFunctionParameters lint rule and fix all violations
+- Enable noUnusedVariables lint rule and remove all unused variables from codebase
+- Replace all var declarations with let/const for modern JavaScript standards
+- Ensure parseInt calls use explicit radix parameter for clarity and reliability
+- Fix precision loss in test data generators by using JavaScript safe integer limits
+- Add block scoping to switch statement cases to prevent variable declaration issues
+- Enforce const usage for variables that are never reassigned
+- Add node: protocol prefix to Node.js builtin module imports for clarity
+- Use modern exponentiation operator (**) instead of Math.pow()
+- Replace string concatenation with modern template literals
+- Remove redundant 'use strict' directives as modules are automatically in strict mode
+- Refactor assignment-in-expression patterns to improve code clarity and readability
+- Replace comma operators with individual statements for clearer, more readable code
+- Split comma-separated variable declarations into individual statements for consistency
+- Replace Object.prototype.hasOwnProperty() with safer Object.hasOwn() method
+- Enforce strict equality checks (=== and !==) instead of loose equality (== and !=)
+- Replace global isNaN with Number.isNaN for safer type checking
+- Ensure no variable redeclarations exist to prevent shadowing issues
+- Replace global isFinite with Number.isFinite for safer numeric validation
+- Enable useArrowFunction lint rule to prefer arrow functions for cleaner syntax
+- Enable noUselessCatch lint rule to prevent useless catch blocks that only rethrow errors
+- Enable noArguments rule to enforce modern rest parameters instead of legacy arguments object
+- Enable bracketSpacing formatting rule to add spaces inside object literals
+- Migrate test suite from Mocha to Node.js built-in test runner
+
+## v0.10.9
+- Add support for IPv6 urls
+
 ## v0.10.8
 - Updated README
 

package/CLAUDE.md

@@ -0,0 +1,12 @@
+# Project-Specific Guidelines for amqplib
+
+## Generated Files - DO NOT EDIT
+- `lib/defs.js` - This file is generated code. NEVER modify it directly.
+  - Generated via `make lib/defs.js` from the AMQP specification
+  - If linting issues arise, exclude this file from linting rules rather than modifying it
+
+## Commit Requirements
+- NEVER commit when tests are failing
+- NEVER commit when there are lint errors
+- Always run `npm test` and `npm run lint` before committing
+- If tests are failing, investigate and fix the issue before proceeding

package/Makefile

@@ -2,12 +2,9 @@ RABBITMQ_SRC_VERSION=v3.12.13
 JSON=amqp-rabbitmq-0.9.1.json
 AMQP_JSON=https://raw.githubusercontent.com/rabbitmq/rabbitmq-server/$(RABBITMQ_SRC_VERSION)/deps/rabbitmq_codegen/$(JSON)
 
-NODEJS_VERSIONS='10.21' '11.15' '12.18' '13.14' '14.5' '15.8' '16.3.0' '18.1.0' '20.10.0' '22.14.0'
+NODEJS_VERSIONS='18.1.0' '20.10.0' '22.14.0' '24.7.0'
 
-MOCHA=./node_modules/.bin/mocha
-_MOCHA=./node_modules/.bin/_mocha
 UGLIFY=./node_modules/.bin/uglifyjs
-NYC=./node_modules/.bin/nyc
 
 .PHONY: test test-all-nodejs coverage lib/defs.js
 
@@ -16,17 +13,16 @@ error:
 	@exit 1
 
 test:
-	$(MOCHA) --check-leaks -u tdd --exit test/
+	node --test
 
 test-all-nodejs:
 	for v in $(NODEJS_VERSIONS); \
 		do echo "-- Node version $$v --"; \
-		nave use $$v $(MOCHA) -u tdd --exit -R progress test; \
+		nave use $$v node --test; \
 		done
 
-coverage: $(NYC)
-	$(NYC) --clean --reporter=lcov --reporter=text $(_MOCHA) -u tdd --exit -R progress test/
-	@echo "HTML report at file://$$(pwd)/coverage/lcov-report/index.html"
+coverage:
+	node --test --test-coverage
 
 lib/defs.js: clean bin/generate-defs test
 
@@ -42,8 +38,5 @@ bin/generate-defs: $(UGLIFY) bin/generate-defs.js bin/amqp-rabbitmq-0.9.1.json
 bin/amqp-rabbitmq-0.9.1.json:
 	curl -L $(AMQP_JSON) > $@
 
-$(ISTANBUL):
-	npm install
-
 $(UGLIFY):
 	npm install