Skip to content

Comments

Feat: add option to configure logging level and format#477

Merged
kobiebotha merged 7 commits intomainfrom
feat/configurable-logging
Jan 29, 2026
Merged

Feat: add option to configure logging level and format#477
kobiebotha merged 7 commits intomainfrom
feat/configurable-logging

Conversation

@kobiebotha
Copy link
Contributor

Why

Solves https://discord.com/channels/1138230179878154300/1463146725832265926

What

  • Moved logger configuration responsibility into service-core
  • Keep logger primitives in lib-services
  • Single shared definition of LoggingConfig in service-types.

Testing performed

  • Tested config permutations by hand using both yaml config and env vars
  • Ran affected automated tests:
pnpm --filter @powersync/lib-services-framework test --run
pnpm --filter @powersync/service-core test --run

- move LoggingConfig declaration to service-types
- implement configureLogger in service-core (doesn't need to stay in
lib-services)
@changeset-bot
Copy link

changeset-bot bot commented Jan 28, 2026

🦋 Changeset detected

Latest commit: 45a82ae

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 17 packages
Name Type
@powersync/service-core Minor
@powersync/lib-services-framework Minor
@powersync/service-types Minor
@powersync/service-core-tests Patch
@powersync/service-module-core Patch
@powersync/service-module-mongodb-storage Patch
@powersync/service-module-mongodb Patch
@powersync/service-module-mssql Patch
@powersync/service-module-mysql Patch
@powersync/service-module-postgres-storage Patch
@powersync/service-module-postgres Patch
@powersync/service-image Minor
test-client Patch
@powersync/service-rsocket-router Patch
@powersync/lib-service-mongodb Patch
@powersync/lib-service-postgres Patch
@powersync/service-schema Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Rentacookie
Rentacookie previously approved these changes Jan 29, 2026
@simolus3
Copy link
Contributor

Note that this needs a changeset entry covering affected packages since service-core depends on changes in lib-services. IIRC, not adding one might lead to us deploying services with inconsistent package versions (which causes confusing crashes).

@Rentacookie Rentacookie self-requested a review January 29, 2026 14:26
@kobiebotha kobiebotha merged commit a02cc58 into main Jan 29, 2026
25 checks passed
@kobiebotha kobiebotha deleted the feat/configurable-logging branch January 29, 2026 18:33
@rkistner rkistner mentioned this pull request Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants