Merge/jan26 01 #33
Merged
Merge/jan26 01 #33
Conversation
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
…ports in index.node.ts; add comprehensive Copilot instructions and build guidelines
- Removed unused peer dependency 'pino-pretty' from peerDependencies. - Cleaned up devDependencies by removing 'barrelsby' and 'typescript-eslint' duplicate entry. - Added '@types/luxon' and '@types/node' to devDependencies for better type support. - Removed the import of '@rollup/plugin-commonjs' from rollup.config.js as it was not being used.
…scope Co-authored-by: barluq <[email protected]>
Co-authored-by: barluq <[email protected]>
Optimize logger by creating single pino instance at module scope
Jan26 extended logging + AI
mbabic84
approved these changes
Jan 9, 2026
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.
This pull request introduces significant improvements to the repository's documentation and codebase, focusing on standardizing coding, testing, and build practices, as well as enhancing logging utilities and utility functions. The most important changes are summarized below:
Documentation and Repository Practices:
AGENTS.md,.github/copilot-instructions.md,.github/instructions/coding.instructions.md,.github/instructions/testing.instructions.md,.github/instructions/build.instructions.md) detailing repository structure, coding standards, testing guidelines, and build strategies. [1] [2] [3] [4]Logging Utilities Overhaul:
AppLoggerclass with a set of functional logging utilities (loggyInfo,loggyDebug,loggyWarn,loggyTrace,loggyFatal,loggyAppStart,loggyRequestReceived,loggyResponseSent) insrc/node/logging/logger.ts, and updated exports insrc/index.node.tsaccordingly. These utilities provide more granular and flexible logging for Node.js applications. [1] [2]Utility Functions Enhancements:
isUrlandisArrayOfUrlsinsrc/globals/coding/code.formating.tsfor URL validation, andnowPlusTimeinsrc/globals/coding/code.dates.tsfor timestamp calculations. Updated exports in both browser and node entrypoints. [1] [2] [3]Build and Dependency Updates:
pino-pretty,barrelsby,@rollup/plugin-commonjs) and updatedpackage.jsonto clean up and clarify peer and dev dependencies. [1] [2] [3] [4]These changes collectively improve maintainability, enforce best practices, and enhance the developer experience across the codebase.