sojoner opened a new pull request, #193:
URL: https://github.com/apache/age-viewer/pull/193
Hello Maintainers,
Not sure if you want this, but I needed this to work for an internal
project. I spend some money on Claude, and got this working for.
As the money is spend and I can now use this in my stack, I though try to PR
this. No worries of there is no time for review, as this is AI generated and
I'm well aware of the stress this puts on maintainers. Feel free to close this.
But maybe it is useful as well. It is useful to me 💯
Best Sojoner
---
This commit addresses critical compatibility and stability issues with
dependency upgrades and includes containerization support.
- Upgrade React 17 → 18, react-scripts 4 → 5 (Webpack 5)
- Update cytoscape and related dependencies to latest versions
- Fix React 18 useEffect lifecycle violation in DefaultTemplate.jsx
* Wrapped async code in useEffect callback to prevent Promise return
* Resolves "destroy is not a function" runtime error
- Improve error handling in DatabaseSlice with try-catch for JSON parsing
- Add default form values (host.docker.internal, rag_user, etc.) for easier
testing
- Disable cytoscape-spread layout to avoid weaverjs dependency issues
- Fix PostgreSQL 18+ compatibility in SQLFlavorManager.js
* Added fallback logic: versions >= 16 use v15 SQL queries
* Fallback cascade for unsupported versions (16-15-14-13-12-11)
- Improve error handling in database connection attempts
- Add credentials: 'include' for session management
- Add Makefile with build automation targets:
* build-node, clear, build-docker, test-docker
* clean-containers, clean-images for Docker management
- Update .gitignore to explicitly exclude backend/build and
frontend/.env.local
✅ Frontend: Builds successfully (903 KB gzipped, linting warnings only)
✅ Backend: Compiles with Babel, unit tests pass
✅ Docker: Image builds and runs with both services functioning
- Integration tests require test database setup (environment issue, not code)
- Linting warnings (no-alert, no-console) are non-blocking and pre-existing
patterns
- Backend audit shows 19 vulnerabilities from dependencies (pre-existing
from version upgrades)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
