Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: microsoft/typespec
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main@{1day}
Choose a base ref
...
head repository: microsoft/typespec
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
  • 14 commits
  • 299 files changed
  • 9 contributors

Commits on Mar 6, 2025

  1. Copy the full SHA
    159beb7 View commit details
  2. Copy the full SHA
    5432e72 View commit details
  3. [http-server-csharp] Fix generation of multiline and long comments (#…

    …6267)
    
    issue: #5465
    ### Example
    ```typespec
    
     @doc("""
        Format of the redacted output. Only valid when Operation is Redact. 
        Please refer to https://learn.microsoft.com/en-us/azure/healthcare-apis/deidentification/redaction-format for more details.
        """)
      redactionFormat?: string;
    ```
    
    
    ## Previously
    ```csharp
      ///<summary>
      /// Format of the redacted output. Only valid when Operation is Redact.  Please
      /// Format of the redacted outpu
      /// Format of the redacted output. Only valid when Operation is Redact.  P
      /// ils.
      ///</summary>
      [TypeSpec.Helpers.JsonConverters.StringConstraint(MinLength = 1, MaxLength = 16)]
      public string RedactionFormat { get; set; }
    ```
    
    ## Now
    ```csharp 
    ///<summary>
    /// Format of the redacted output. Only valid when Operation is Redact.  Please
    /// refer to
    /// https://learn.microsoft.com/en-us/azure/healthcare-apis/deidentification/redaction-format
    /// for more details.
    ///</summary>
      [StringConstraint( MinLength = 1, MaxLength = 16)]
      public string RedactionFormat { get; set; }
    ```
    AlitzelMendez authored Mar 6, 2025
    Copy the full SHA
    b8ca102 View commit details
  4. Copy the full SHA
    a7b8376 View commit details
  5. update workspace version range (#6303)

    Fixes #1848 
    
    Link to typespec azure changes:
    Azure/typespec-azure#2322
    
    ---------
    
    Co-authored-by: Christopher Radek <Christopher.Radek@microsoft.com>
    chrisradek and Christopher Radek authored Mar 6, 2025
    Copy the full SHA
    5d93e10 View commit details
  6. Convert all compiler export * to named export (#6294)

    First step in figuring out what should be internal, this will also help
    in the future to prevent exposing new types by mistake
    
    This convert the export * to named exports so we can get a manifest of
    what is exported. **This pr shouldn't remove anything**
    
    ---------
    
    Co-authored-by: Mark Cowlishaw <markcowl@microsoft.com>
    timotheeguerin and markcowl authored Mar 6, 2025
    Copy the full SHA
    37b0646 View commit details
  7. Fix path for emitted files (#6309)

    The relative wasn't correct when nested folder in the path. I was
    overprocessing the path. Fixing this
    AlitzelMendez authored Mar 6, 2025
    Copy the full SHA
    d211084 View commit details
  8. Removes deprecated isContentTypeHeader usage from openapi3 (#6311)

    Co-authored-by: Christopher Radek <Christopher.Radek@microsoft.com>
    chrisradek and Christopher Radek authored Mar 6, 2025
    Copy the full SHA
    15cb64a View commit details

Commits on Mar 7, 2025

  1. Download configured package manager and run it (#6220)

    fix #5687
    
    Features
    - respect the `packageManager`(Format `<packageManager>@<version>` or
    `<packageManager>@<version>+<algo>.<hash>`) and
    `devEngines.packageManager` field
    - packages are be downloaded from the public npm registry
    -no support yet for proxy, private registries, using the node route is
    always available for those cases
    - ability to save a hash of the packagemanager in the field which is
    used to validate the `.tgz` file match.
    - new flag `--save-package-manager` which populate the `packageManager`
    field with the resolved version and the hash.
    
    
    Only support `npm` for now. Adding support for `yarn` and `pnpm` should
    be quite simple.
    
    ---------
    
    Co-authored-by: Christopher Radek <14189820+chrisradek@users.noreply.github.com>
    timotheeguerin and chrisradek authored Mar 7, 2025
    Copy the full SHA
    3fb34c1 View commit details
  2. Upgrade alloy dependency (#6308)

    Upgrade to alloy 0.6.0
    
    Impacted packages:
    @typespec/http-client
    @typespec/emitter-framework
    @typespec/http-client-js
    
    Changes are around formatting leveraging the new formatting capabilities
    from allo such as <For>, <StatementList>, <List>
    joheredi authored Mar 7, 2025
    Copy the full SHA
    1edb744 View commit details
  3. http-client-java, fix duplicate mock test (#6298)

    We'll truncate mock test class name when the file path is too long. This
    will result in a duplicate file path in edge cases:
    
    https://dev.azure.com/azure-sdk/internal/_build/results?buildId=4623919&view=logs&j=83516c17-6666-5250-abde-63983ce72a49&t=c10d5f44-55ce-55d7-7975-407ed75d9a96&l=40534
    XiaofeiCao authored Mar 7, 2025
    Copy the full SHA
    f18d636 View commit details
  4. Copy the full SHA
    5df4498 View commit details
  5. http-client-java, improve error diagnostics for spotless failure (#6300)

    The failure is usually due to Java syntax problem. We want to print the
    file name and content for better diagnostics.
    
    Sample output of this PR.
    
    ```
    [main] ERROR com.microsoft.typespec.http.client.generator.TypeSpecPlugin - Failed to format file 'src/main/java/azure/clientgenerator/core/usage/implementation/ModelInOperationsImpl.java'
    177:15: error: illegal unicode escape
    174 
    175     /**
    176      * Serialize the 'OrphanModel' as request body.
    177      * data\userdomain
    178      * 
    179      * Expected body parameter: 
    180      * ```json
    ```
    
    I've also add logic to parse the JAR output to diagnostics/trace for
    tsp.
    
    
    ![image](https://github.com/user-attachments/assets/180cebb0-de05-4094-8820-7d34928d7615)
    
    
    ![image](https://github.com/user-attachments/assets/4b8b12e9-1bad-489e-bdeb-93ff485f9347)
    
    autorest.java Azure/autorest.java#3035
    weidongxu-microsoft authored Mar 7, 2025
    Copy the full SHA
    94a9b9e View commit details
  6. rest - filter string constant path http properties when using `@autoR…

    …oute` (#6326)
    
    #6311 switched openapi3 to use `HttpProperty` instead of
    `HttpOperationParameter`. The `@typespec/rest` package did some
    filtering of parameters under a couple conditions (including path
    constants) that wasn't applied to properties. This caused an extra path
    parameter to be emitted in OpenAPI3 (and typespec-autorest when
    eventually applying the same changes there).
    
    This PR applies the same `@autoRoute` parameter filtering to properties.
    
    ---------
    
    Co-authored-by: Christopher Radek <Christopher.Radek@microsoft.com>
    chrisradek and Christopher Radek authored Mar 7, 2025
    Copy the full SHA
    7a77cf4 View commit details

This comparison is taking too long to generate.

Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.

You can try running this command locally to see the comparison on your machine:
git diff main@{1day}...main