Conversation
|
This has bee rewritten in various goals using Codex and GPT 5.6 Sol. Going to start reviewing the code all and all. |
b7d0b6f to
33eac53
Compare
56bff68 to
8f8bec6
Compare
|
These security improvements and features sound great but was there a need for such a massive architecture shift? It seems quite tiresome to review a PR which quintuples the line count of the repository. More than that though it makes me quite nervous that something would break silently, especially since Gamma is such a central part of the division's systems |
144ee08 to
929284e
Compare
|
@olillin I'm confident in the result of this rewrite. And I'm confident that it will lead to better Gamma. It should however be in draft still, I'm starting to land on an architecture I like. I will also deploy 2.5.1 first on railway, along with chalmers.it and bookit. Then, I will upgrade to 2.6.0 to make sure the migration is smooth. |
10faf42 to
51a97a0
Compare
Retain typed Exposed queries while making transaction ownership, cross-context participation, and external effects explicit. Remove forwarding layers, fix allow-list commit outcomes, and cover operation guarantees with PostgreSQL integration tests. Verified with 585 JVM tests, 54 Chromium browser tests, full lint and formatting checks, context boundaries, coverage thresholds, and a production image build.
|
I will close this PR. I had a lot of fun working through many different refactoring, but I think for the best of the project that I keep the incrementing changes instead of the mega rewrite. For the record, I have full confidence that this rewrite would work, but I do not think it is the best way forward regardless. |
2 participants
No description provided.