From b10e1bb5008a9d856c382d550ba4e6d4a444c1f4 Mon Sep 17 00:00:00 2001 From: Merango Date: Thu, 22 May 2025 21:11:02 +0000 Subject: [PATCH 1/6] Start draft PR From 0a8f785da24d856843da4d1d0d9fc26ab74b63f6 Mon Sep 17 00:00:00 2001 From: Merango Date: Thu, 22 May 2025 21:11:21 +0000 Subject: [PATCH 2/6] Update package.json with modern dependencies and scripts --- package.json | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/package.json b/package.json index 5aa2568..ca82f7b 100644 --- a/package.json +++ b/package.json @@ -24,19 +24,25 @@ "needle": "^1.6.0" }, "devDependencies": { - "jscs": ">=3.0.2", - "nodeunit": "0.11.3" + "@types/node": "^16.18.0", + "eslint": "^8.45.0", + "eslint-config-recommended": "^4.1.0", + "nodeunit": "^0.11.3", + "typescript": "^5.1.6" }, "scripts": { - "test": "node ./node_modules/.bin/nodeunit test" + "test": "node ./node_modules/.bin/nodeunit test", + "lint": "eslint .", + "lint:fix": "eslint . --fix", + "prepare": "npm run lint" }, "license": "MIT", - "main": "index", + "main": "index.js", "engines": { - "node": ">= 0.8.0" + "node": ">= 14.0.0" }, "readmeFilename": "Readme.md", "bugs": { "url": "https://github.com/rchipka/node-osmosis/issues" } -} +} \ No newline at end of file From 85abd6c190385f9936b96b6cd321f1e39ec81f2e Mon Sep 17 00:00:00 2001 From: Merango Date: Thu, 22 May 2025 21:11:27 +0000 Subject: [PATCH 3/6] Add ESLint configuration for code quality --- .eslintrc.json | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) create mode 100644 .eslintrc.json diff --git a/.eslintrc.json b/.eslintrc.json new file mode 100644 index 0000000..29f47c7 --- /dev/null +++ b/.eslintrc.json @@ -0,0 +1,22 @@ +{ + "env": { + "node": true, + "es2021": true, + "mocha": true + }, + "extends": [ + "eslint:recommended" + ], + "parserOptions": { + "ecmaVersion": 2021, + "sourceType": "module" + }, + "rules": { + "indent": ["error", 2], + "linebreak-style": ["error", "unix"], + "quotes": ["error", "single"], + "semi": ["error", "always"], + "no-console": "warn", + "no-unused-vars": "warn" + } +} \ No newline at end of file From 67ba21bb4bea881026ad5c18016453236814aac6 Mon Sep 17 00:00:00 2001 From: Merango Date: Thu, 22 May 2025 21:11:34 +0000 Subject: [PATCH 4/6] Update .gitignore with comprehensive ignore patterns --- .gitignore | 45 +++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 43 insertions(+), 2 deletions(-) diff --git a/.gitignore b/.gitignore index 74ca64e..db25d4e 100644 --- a/.gitignore +++ b/.gitignore @@ -1,3 +1,44 @@ +# Dependency directories node_modules/ -docs/ -npm-debug.log + +# Optional npm cache directory +.npm + +# Optional eslint cache +.eslintcache + +# Optional REPL history +.node_repl_history + +# Output of 'npm pack' +*.tgz + +# Logs +logs +*.log +npm-debug.log* +yarn-debug.log* +yarn-error.log* + +# Temporary files +.tmp +.temp + +# IDE settings +.vscode/ +.idea/ + +# Environment files +.env +.env.local +.env.development.local +.env.test.local +.env.production.local + +# Mac files +.DS_Store +.AppleDouble +.LSOverride + +# Coverage directory used by tools like istanbul +coverage/ \ No newline at end of file From 3e98762bc4127332971c210b62536003c5717415 Mon Sep 17 00:00:00 2001 From: Merango Date: Thu, 22 May 2025 21:12:15 +0000 Subject: [PATCH 5/6] Update package dependencies for modern Node.js compatibility --- package.json | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/package.json b/package.json index ca82f7b..e5d5a2c 100644 --- a/package.json +++ b/package.json @@ -20,26 +20,23 @@ "email": "chipka01@email.franklin.edu" }, "dependencies": { - "libxmljs-dom": "~0.0.17", - "needle": "^1.6.0" + "libxmljs-dom": "^0.18.4", + "needle": "^3.2.0" }, "devDependencies": { - "@types/node": "^16.18.0", + "@types/node": "^18.16.0", "eslint": "^8.45.0", - "eslint-config-recommended": "^4.1.0", - "nodeunit": "^0.11.3", - "typescript": "^5.1.6" + "nodeunit": "^0.11.3" }, "scripts": { "test": "node ./node_modules/.bin/nodeunit test", "lint": "eslint .", - "lint:fix": "eslint . --fix", - "prepare": "npm run lint" + "lint:fix": "eslint . --fix" }, "license": "MIT", "main": "index.js", "engines": { - "node": ">= 14.0.0" + "node": ">=14.0.0" }, "readmeFilename": "Readme.md", "bugs": { From a0a3ddffd00d7be4ab4015b1aa049d7a71ce1597 Mon Sep 17 00:00:00 2001 From: Merango Date: Thu, 22 May 2025 21:12:29 +0000 Subject: [PATCH 6/6] Add custom test runner script for nodeunit tests --- run-tests.js | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) create mode 100644 run-tests.js diff --git a/run-tests.js b/run-tests.js new file mode 100644 index 0000000..fce1e22 --- /dev/null +++ b/run-tests.js @@ -0,0 +1,17 @@ +const nodeunit = require('nodeunit'); +const fs = require('fs'); +const path = require('path'); + +const testFiles = fs.readdirSync('test') + .filter(file => file.endsWith('.js') && file !== 'server/index.js') + .map(file => path.join('test', file)); + +nodeunit.reporters.default.run(testFiles, null, (err) => { + if (err) { + console.error('Tests failed:', err); + process.exit(1); + } else { + console.log('All tests passed successfully'); + process.exit(0); + } +}); \ No newline at end of file