Improve CI debuggability with structured test logging#22
Merged
Conversation
Change the mise log_level from debug to trace in the setup-mise action, and add MISE_LOG_LEVEL environment variable to the run steps for both the check and test jobs. This ensures mise outputs more detailed logs when runner.debug is enabled, helping diagnose CI issues.
Add LogTape logging throughout the application stack: - CLI: --log-level/-L and --log-output options via @optique/logtape integration (loggingOptions with level: "option") - Database: Drizzle ORM queries logged via @logtape/drizzle-orm - GraphQL Yoga: request/response logging via @logtape/graphql-yoga - Server: configure() with console sink at startup in packages/drfed/src/index.ts Update the dev task in mise.toml and scripts/dev.mts to pass --log-level (defaulting to "debug") and --log-output through to the development server process.
Replace all node:test describe/it imports with @logtape/testing-node/autoload, which integrates log output with the test runner for better debugging. Update @logtape/logtape and @logtape/graphql-yoga to 2.3.0-dev.840 to match the testing-node dependency. Add LOGTAPE_TEST_MODE and LOGTAPE_TEST_LOWEST_LEVEL environment variables to both CI workflow and local mise test task to control when and at what level log output is shown during tests.
Replace static environment variables with a Nushell script that supports a --debug flag. When the flag is set, log output is shown at trace level always; otherwise it defaults to showing logs on failure at debug level.
sij411
approved these changes
Jul 3, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Replace
node:test'sdescribe/itwith@logtape/testing-node/autoloadso test failures include structured log output. Upgrades@logtape/logtapeand@logtape/graphql-yogato matching dev builds.Adds
LOGTAPE_TEST_MODEandLOGTAPE_TEST_LOWEST_LEVELto CI (GitHub Actions) and localmise run test, plus a-d/--debugflag that shows trace-level logs on every test run.