-
Notifications
You must be signed in to change notification settings - Fork 626
Postgres #5365
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Postgres #5365
Changes from 61 commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
de730f5
postgres experimental WIP
boutell dccb36d
astonishingly, all mocha tests of apostrophe pass with this
boutell 266841c
mocha tests pass, actual sites work
boutell 9d67565
lint clean
boutell 04d4edd
listDatabases support, but changes are coming
boutell bc7b72b
wip
boutell 9ba57fc
dump and restore updates
boutell 5ad412a
backpressure, adequate handling of ObjectId for our needs (becomes it…
boutell db1434b
mild performance optimization
boutell a083c25
profiling
boutell ddf80df
testing issue resolved
boutell 3c7a23b
refactored to db-connect module, introduced sqlite adapter
boutell 8b095f1
sqlite WIP
boutell 80dd7ab
debugging
d08f502
programmatic API for dump/restore/copy dbs
374b5d8
linting, documentation
boutell 6a5b931
MIT license
boutell fea05a7
text ranking is more accurate, documentation is more complete
boutell b67e2cd
good full text search for sqlite
boutell 54dc9b5
updates for compatibility with the rest of the public and private mod…
boutell f0f45f9
requirements found by testing private modules
boutell f4ae5d8
Merge branch 'main' into postgres
boutell 9c9105d
fixes from full cypress run
boutell ab875ef
eslint passing
boutell c95c9b3
restore permissions
boutell 88e8e48
maximize atomicity
boutell 2eadf8e
bug fixes
boutell 9494c00
* exit properly when asset tests fail
boutell d3090df
ignore claude-tools in eslint
boutell 5b3c675
postgres and sqlite-inclusive ci matrix attempt
boutell fe2cc10
clean up logs
boutell 50897fd
We hit github's limit on total configurations because every package g…
boutell 3c29e2a
hardened the asset tests, made them less timing sensitive, fixed a ba…
boutell 1370762
fix a root cause of asset test instability
boutell 4b1716e
log mess
boutell 2fc2224
implemented missing $size operator
boutell c260229
test compatibility
boutell 7408e4b
advanced permission uses regex in $in
boutell f0bfb0e
regex in $in
boutell 1f1e6b3
.db() should not make false promises in plain postgres mode, it shoul…
boutell c198a01
ability to specify a default adapter
boutell 14dffe3
obsolete file
boutell 7e60e35
put escapeHost back where it belongs
boutell bef7978
dead code removal, test cleanup
boutell 8cb3c5a
emulate-mongo-3-driver only needed in db-connect
boutell ec9ec17
no claude logs in repo (tools are welcome)
boutell 0283d85
* shared aggregation implementation, other shared things
boutell 3a8901d
vanilla postgres should not attempt to use .db() with alternate names…
boutell 061136a
documentation corrections
boutell 9c91f9f
documentation errors
boutell 4e11aad
listDatabases and documentation corrections
boutell 615399f
Merge branch 'main' into postgres
boutell f492eb4
more edge cases revealed by latest work from Miro
boutell c7afe8d
anchored prefix regexps are optimized
boutell 7aa5fd1
* matchesQuery in the aggregation cursor implementation doesn't thr…
boutell cfdd3c6
do not swallow dump/restore errors on indexes
boutell a9338c4
cover how to run the utilities
boutell c863da6
fix detection of source
boutell 6971100
separate sanitization for index names
boutell 3885742
Merge branch 'main' into postgres
boutell 8bc9d3f
regex prefix safety
boutell 9af8358
pnpm
boutell File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,3 +6,5 @@ public/apos-frontend | |
| .DS_Store | ||
| coverage/ | ||
| .nyc_output | ||
| claude-tools/logs/ | ||
| .claude | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,37 @@ | ||
| #!/bin/bash | ||
| # Run the apostrophe core test suite against a chosen DB adapter and log | ||
| # output to claude-tools/logs/core-<adapter>.log. Usage: | ||
| # | ||
| # ./claude-tools/run-core-tests.sh mongodb | ||
| # ./claude-tools/run-core-tests.sh postgres | ||
| # ./claude-tools/run-core-tests.sh sqlite | ||
| # | ||
| # NEVER run multiple adapters in parallel — the test suite is not designed | ||
| # for concurrent runs and the host has limited resources. | ||
|
|
||
| set -u | ||
| adapter="${1:-}" | ||
| if [[ -z "$adapter" ]]; then | ||
| echo "usage: $0 <mongodb|postgres|sqlite>" >&2 | ||
| exit 2 | ||
| fi | ||
|
|
||
| root="$(cd "$(dirname "$0")/.." && pwd)" | ||
| logdir="$root/claude-tools/logs" | ||
| mkdir -p "$logdir" | ||
| log="$logdir/core-$adapter.log" | ||
| : > "$log" | ||
|
|
||
| echo "=== $adapter core tests ($(date -Is)) ===" | tee -a "$log" | ||
|
|
||
| cd "$root/packages/apostrophe" | ||
|
|
||
| extra=() | ||
| if [[ "$adapter" == "postgres" ]]; then | ||
| extra=(env PGPASSWORD=testpassword) | ||
| fi | ||
|
|
||
| APOS_TEST_DB_PROTOCOL="$adapter" "${extra[@]}" npm run test:base >> "$log" 2>&1 | ||
| code=$? | ||
| echo "=== exit=$code ===" | tee -a "$log" | ||
| exit "$code" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -42,3 +42,6 @@ test/public/uploads | |
|
|
||
| # vim swp files | ||
| .*.sw* | ||
|
|
||
| # claude-tools log files | ||
| claude-tools/**/*.log | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. claude-tools is meant to persist, but its logs are not |
||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be probably
pnpminsteadnpm.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed that.