Skip to content
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

Bruno #4

Open
wants to merge 2 commits into
base: readme
Choose a base branch
from
Open

Bruno #4

wants to merge 2 commits into from

Conversation

Chetan-Rayalu
Copy link
Collaborator

@Chetan-Rayalu Chetan-Rayalu commented Feb 18, 2025

PR Type

dependencies, documentation


Description

  • Added multiple JSON files containing character encoding mappings for CP936, CP950, EUC-JP, and other encodings.

  • Introduced TypeScript declaration files for utilities like Reflect.apply, Function.prototype.apply, and Object.getOwnPropertyDescriptor.

  • Added documentation files, including a README for the http-errors module and changelogs for serve-static and other packages.

  • Implemented a new JavaScript module (depd) for deprecation handling, including utilities for logging and stack trace manipulation.


Changes walkthrough 📝

Relevant files
Dependencies
7 files
cp936.json
Added CP936 character encoding table for iconv-lite.         

node_modules/iconv-lite/encodings/tables/cp936.json

  • Added a JSON file containing character encoding mappings for CP936.
  • Includes a large array of Unicode character mappings.
  • +264/-0 
    actualApply.d.ts
    Added TypeScript declaration for Reflect.apply.                   

    node_modules/call-bind-apply-helpers/actualApply.d.ts

    • Added a TypeScript declaration file exporting Reflect.apply.
    +1/-0     
    cp950.json
    Added Unicode character mapping table for cp950 encoding.

    node_modules/iconv-lite/encodings/tables/cp950.json

  • Added a JSON file containing a large array of Unicode character
    mappings.
  • Includes mappings for various character ranges and encodings.
  • +177/-0 
    index.js
    Added depd module for deprecation warnings and handling. 

    node_modules/depd/index.js

  • Introduced a new JavaScript module for deprecation handling.
  • Includes functions for logging, wrapping functions and properties, and
    managing namespaces.
  • Implements utilities for stack trace manipulation and error creation.
  • +538/-0 
    gOPD.d.ts
    Added TypeScript declaration for gOPD utility.                     

    node_modules/gopd/gOPD.d.ts

  • Added a TypeScript declaration file exporting
    Object.getOwnPropertyDescriptor.
  • +1/-0     
    eucjp.json
    Add EUC-JP encoding tables for character mapping.               

    node_modules/iconv-lite/encodings/tables/eucjp.json

  • Added a JSON file containing EUC-JP encoding tables.
  • Includes mappings for various character ranges and their Unicode
    equivalents.
  • Provides support for encoding and decoding Japanese characters.
  • +182/-0 
    functionApply.d.ts
    Add TypeScript declaration for Function.prototype.apply. 

    node_modules/call-bind-apply-helpers/functionApply.d.ts

  • Added a TypeScript declaration file exporting
    Function.prototype.apply.
  • Provides type definition for usage of apply method in helper
    functions.
  • +1/-0     
    Documentation
    2 files
    README.md
    Added documentation for the http-errors module.                   

    node_modules/http-errors/README.md

  • Added a README file for the http-errors module.
  • Includes installation instructions, API documentation, and usage
    examples.
  • Provides a list of HTTP error constructors and their corresponding
    status codes.
  • +169/-0 
    HISTORY.md
    Add changelog for serve-static package versions.                 

    node_modules/serve-static/HISTORY.md

  • Added a changelog file documenting the history of changes for the
    serve-static package.
  • Includes version updates, dependency changes, and notable fixes or
    features.
  • Covers changes from version 1.0.0 to 1.16.2.
  • +487/-0 
    Additional files
    101 files
    Testing.bru +11/-0   
    bruno.json +9/-0     
    mime +16/-0   
    mime.cmd +17/-0   
    mime.ps1 +28/-0   
    .package-lock.json +825/-0 
    HISTORY.md +243/-0 
    LICENSE +23/-0   
    README.md +140/-0 
    index.js +238/-0 
    package.json +47/-0   
    LICENSE +21/-0   
    README.md +43/-0   
    array-flatten.js +64/-0   
    package.json +39/-0   
    HISTORY.md +672/-0 
    LICENSE +23/-0   
    README.md +476/-0 
    SECURITY.md +25/-0   
    index.js +156/-0 
    read.js +205/-0 
    json.js +247/-0 
    raw.js +101/-0 
    text.js +121/-0 
    urlencoded.js +307/-0 
    package.json +56/-0   
    History.md +97/-0   
    LICENSE +23/-0   
    Readme.md +152/-0 
    index.js +170/-0 
    package.json +42/-0   
    .eslintrc +16/-0   
    FUNDING.yml +12/-0   
    .nycrc +9/-0     
    CHANGELOG.md +23/-0   
    LICENSE +21/-0   
    README.md +62/-0   
    actualApply.js +10/-0   
    applyBind.d.ts +19/-0   
    applyBind.js +10/-0   
    functionApply.js +4/-0     
    functionCall.d.ts +1/-0     
    functionCall.js +4/-0     
    index.d.ts +46/-0   
    index.js +15/-0   
    package.json +85/-0   
    reflectApply.d.ts +3/-0     
    reflectApply.js +4/-0     
    index.js +63/-0   
    tsconfig.json +9/-0     
    .eslintrc +13/-0   
    FUNDING.yml +12/-0   
    .nycrc +9/-0     
    CHANGELOG.md +34/-0   
    LICENSE +21/-0   
    README.md +53/-0   
    index.d.ts +13/-0   
    index.js +18/-0   
    package.json +99/-0   
    index.js +54/-0   
    tsconfig.json +9/-0     
    HISTORY.md +60/-0   
    LICENSE +22/-0   
    README.md +142/-0 
    index.js +458/-0 
    package.json +44/-0   
    HISTORY.md +29/-0   
    LICENSE +22/-0   
    README.md +94/-0   
    index.js +225/-0 
    package.json +42/-0   
    .npmignore +4/-0     
    History.md +38/-0   
    Readme.md +42/-0   
    index.js +51/-0   
    package.json +18/-0   
    LICENSE +24/-0   
    README.md +317/-0 
    SECURITY.md +25/-0   
    index.js +334/-0 
    package.json +44/-0   
    .coveralls.yml +1/-0     
    .eslintrc +11/-0   
    .npmignore +9/-0     
    .travis.yml +14/-0   
    CHANGELOG.md +362/-0 
    LICENSE +19/-0   
    Makefile +50/-0   
    README.md +312/-0 
    component.json +19/-0   
    karma.conf.js +70/-0   
    node.js +1/-0     
    package.json +49/-0   
    browser.js +185/-0 
    debug.js +202/-0 
    index.js +10/-0   
    inspector-log.js +15/-0   
    node.js +248/-0 
    History.md +103/-0 
    LICENSE +22/-0   
    Additional files not shown

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • Copy link

    Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 PR contains tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Error Handling

    The test file adds error handling tests but does not verify error stack traces or error inheritance chain. Consider adding tests for these error properties.

    test('type error', function (t) {
        t.plan(1);
        var aerr = new TypeError();
        aerr.foo = 555;
        aerr.bar = [1, 2, 3];
    
        var berr = new TypeError('tuv');
        berr.baz = 555;
    
        var cerr = new SyntaxError();
        cerr.message = 'whoa';
        cerr['a-b'] = 5;
    
        var withCause = new ErrorWithCause('foo', { cause: 'bar' });
        var withCausePlus = new ErrorWithCause('foo', { cause: 'bar' });
        withCausePlus.foo = 'bar';
        var withUndefinedCause = new ErrorWithCause('foo', { cause: undefined });
        var withEnumerableCause = new Error('foo');
        withEnumerableCause.cause = 'bar';
    
        var obj = [
            new TypeError(),
            new TypeError('xxx'),
            aerr,
            berr,
            cerr,
            withCause,
            withCausePlus,
            withUndefinedCause,
            withEnumerableCause
        ];
        t.equal(inspect(obj), '[ ' + [
            '[TypeError]',
            '[TypeError: xxx]',
            '{ [TypeError] foo: 555, bar: [ 1, 2, 3 ] }',
            '{ [TypeError: tuv] baz: 555 }',
            '{ [SyntaxError: whoa] message: \'whoa\', \'a-b\': 5 }',
            'cause' in Error.prototype ? '[Error: foo]' : '{ [Error: foo] [cause]: \'bar\' }',
            '{ [Error: foo] ' + ('cause' in Error.prototype ? '' : '[cause]: \'bar\', ') + 'foo: \'bar\' }',
            'cause' in Error.prototype ? '[Error: foo]' : '{ [Error: foo] [cause]: undefined }',
            '{ [Error: foo] cause: \'bar\' }'
        ].join(', ') + ' ]');
    });

    Copy link

    Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Impact
    Possible issue
    Breaking change in URL-encoded parsing

    The default depth level for parsing URL-encoded data has been changed from
    Infinity to 32 in version 4.20.0. This could cause issues with deeply nested
    objects. Consider documenting this breaking change and testing affected
    endpoints.

    node_modules/express/History.md [30-32]

     * deps: [email protected]
       * add `depth` option to customize the depth level in the parser
       * IMPORTANT: The default `depth` level for parsing URL-encoded data is now `32` (previously was `Infinity`)
    +  * BREAKING CHANGE: Applications relying on deeply nested URL-encoded data (>32 levels) must explicitly set the `depth` option
    • Apply this suggestion
    Suggestion importance[1-10]: 8

    __

    Why: The suggestion highlights a critical breaking change in body-parser that could silently fail for applications with deeply nested URL-encoded data, potentially causing data loss or functionality issues.

    Medium
    • More

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    1 participant