Tags: tata1mg/catalyst-core
Tags
Sentry SDK (#77) This PR adds a sentry SDK --- --- **DeputyDev generated PR summary:** --- **Size L:** This PR changes include 140 lines and should take approximately 1-3 hours to review --- This pull request adds Sentry integration to the project. Here's a breakdown of the changes: 1. **package.json**: - Updated the version from `0.0.1-beta.14` to `0.0.1-beta.15`. - Added an entry point for Sentry in the exports. 2. **loadEnvironmentVariables.js**: - Added code to load Sentry configuration from a `sentry.config.json` file into environment variables. 3. **sentry.js**: - Introduces a new Sentry integration module. - Contains functions to initialize Sentry, capture exceptions, capture messages, and add breadcrumbs. - Implements an `ErrorBoundary` React component to catch errors in React components. 4. **webpack/base.babel.js**: - Ensures `SENTRY_CONFIG` is available in the client environment. This setup allows the application to use Sentry for error tracking both on the server and in the React front end, depending on the execution environment. --- DeputyDev generated PR summary until f87cc4b
Handler Promise + Header sent bug fix (#60) This PR includes 1. Returning of a promise from handler 2. Fix for header sent error --- --- **DeputyDev generated PR summary:** --- **Size L:** This PR changes include 170 lines and should take approximately 1-3 hours to review --- The pull request addresses two main issues: 1. **Promise Handling**: The PR introduces proper promise handling in the server-side rendering process. It wraps several asynchronous operations in promises to ensure that the execution flow is correctly managed, particularly in the `renderMarkUp` and main handler functions. This approach allows for better error handling and ensures that the response is not prematurely ended or headers are not sent multiple times. 2. **Header Sent Bug Fix**: The PR adds checks to see if headers have already been sent before proceeding with certain operations. This prevents attempts to modify headers or send additional responses after a response has already started, which would otherwise lead to errors. Key changes include: - Wrapping operations in promises to handle asynchronous execution flow. - Adding checks for `res.headersSent` to avoid modifying headers or sending responses after they have already been sent. - Improved error logging to provide more detailed information about failures in the rendering and data-fetching processes. These changes enhance the robustness and reliability of the server-side rendering, particularly in error scenarios. --- DeputyDev generated PR summary until 344ee2d --------- Co-authored-by: Mayan Sharma <mayansharma@G206F1WHVF.local>
Update router version to v7 (#48) --- **DeputyDev generated PR summary:** --- **Size S:** This PR changes include 16 lines and should take approximately 15-30 minutes to review --- The pull request updates the version of the `@tata1mg/router` package from `0.0.1-beta.6` to `0.0.1-beta.7` in the `package.json` file. Additionally, it also updates the version of the `catalyst-core` package itself from `0.0.1-beta.12` to `0.0.1-beta.13`. This indicates that the changes are primarily focused on upgrading the router to a newer version, which might include bug fixes, performance improvements, or new features introduced in the `@tata1mg/router` package. --- DeputyDev generated PR summary until a19544d Co-authored-by: Mayan Sharma <mayansharma@G206F1WHVF.local>
Support for ESM packages (#43) This PR adds support for ESM only packages - Packages that are ESM only were unable to run on catalyst as it is built to cjs and is unable to import those packages, this PR introduces a new `transpileModules` list that can be defined in `webpackConfig.js` to force bundling the packages. ``` // webpackConfig.js { transpileModules: [] } ``` --------- Co-authored-by: utkarsh-1mg <utkarsh.srivastava@1mg.com> Co-authored-by: Mayan Sharma <mayansharma@C02HFD2GQ6L4.local> Co-authored-by: Utkarsh Srivastava <utkarsh.srivastava1306@gmail.com> Co-authored-by: Mayan Sharma <mayansharma@G206F1WHVF.local>
PreviousNext