Bump flatted from 3.3.3 to 3.4.2 in /dotnet/samples/Demos/ProcessFrameworkWithSignalR/src/ProcessFramework.Aspire.SignalR.ReactFrontend#13700
Conversation
Bumps [flatted](https://github.com/WebReflection/flatted) from 3.3.3 to 3.4.2. - [Commits](WebReflection/flatted@v3.3.3...v3.4.2) --- updated-dependencies: - dependency-name: flatted dependency-version: 3.4.2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Automated Code Review
Reviewers: 4 | Confidence: 92%
✓ Correctness
This PR only modifies auto-generated lockfiles (package-lock.json and yarn.lock). The changes remove 'peer: true' flags from packages that are direct or dev dependencies in package.json (react, react-dom,
@fluentui/react-components,@types/react,@types/react-dom, typescript, eslint, vite, etc.), which is correct. The only version change is flated 3.3 → 3.4.2, a patch-level bump of a transitive dev dependency. No correctness issues found.
✓ Security Reliability
This PR updates lockfiles (package-lock.json and yarn.lock) for a React frontend sample app. The changes are: (1) removing
"peer": trueflags from numerous packages, which means these packages are now treated as direct/transitive dependencies rather than peer dependencies — this is consistent with them already being listed in package.json's dependencies/devDependencies, and (2) bumpingflattedfrom 3.3.3 to 3.4.2, a minor version update of a dev dependency used by flat-cache/eslint. No security or reliability concerns identified. The integrity hashes are present and match the expected format for the declared versions. No new dependencies are introduced, no secrets are exposed, and no code changes are involved.
✓ Test Coverage
This PR modifies only lockfiles (package-lock.json and yarn.lock) for a React frontend demo sample. The changes remove
"peer": trueflags from numerous dependencies (promoting them from peer to direct/transitive dependencies) and bumpflatedfrom 3.3.3 to 3.4.2. These are purely dependency metadata changes in lockfiles within a demo/sample project — no application code, library code, or test code is added or modified. There is no new or changed behavior that would require test coverage.
✓ Design Approach
This PR is a lockfile-only update: it removes stale
"peer": trueannotations from packages that are explicitly listed as direct dependencies in package.json (react, react-dom,@fluentui/react-components,@types/react,@types/react-dom, etc.), and bumpsflattedfrom 3.3.3 → 3.4.2 in both lock files. The"peer": trueremovals are correct — those packages genuinely belong todependencies/devDependenciesin package.json, so npm should not mark them as peer-only. Theflattedbump is a minor dev-dependency patch. One pre-existing structural concern (not introduced by this PR): the project maintains bothpackage-lock.jsonandyarn.locksimultaneously, which means the two lock files can diverge silently depending on which package manager CI or developers happen to run. This PR's yarn.lock and package-lock.json updates are consistent with each other, so it does not make the divergence problem worse, but the root cause remains unaddressed.
Suggestions
- The project ships both package-lock.json and yarn.lock. Relying on two competing lock files is a latent correctness risk — whichever package manager a developer or CI pipeline uses may produce different installs. Consider picking one (npm or yarn) and removing the other lock file to eliminate this ambiguity.
Automated review by dependabot[bot]'s agents
### Motivation and Context Semantic Kernel currently has 43 open Dependabot pull requests spanning GitHub Actions, .NET, Python, npm, and Yarn dependency graphs. This rollup applies their compatible net changes together on the latest `main`, making it possible to validate and merge the updates as one coherent dependency state. Supersedes: - microsoft#13133 - microsoft#13134 - microsoft#13136 - microsoft#13172 - microsoft#13507 - microsoft#13601 - microsoft#13606 - microsoft#13666 - microsoft#13673 - microsoft#13688 - microsoft#13700 - microsoft#13707 - microsoft#13708 - microsoft#13712 - microsoft#13721 - microsoft#13873 - microsoft#13930 - microsoft#13939 - microsoft#13949 - microsoft#13950 - microsoft#13951 - microsoft#13997 - microsoft#14000 - microsoft#14053 - microsoft#14069 - microsoft#14088 - microsoft#14091 - microsoft#14094 - microsoft#14103 - microsoft#14106 - microsoft#14107 - microsoft#14108 - microsoft#14109 - microsoft#14110 - microsoft#14128 - microsoft#14148 - microsoft#14149 - microsoft#14160 - microsoft#14161 - microsoft#14164 - microsoft#14171 - microsoft#14172 - microsoft#14173 ### Description - Updates the requested GitHub Actions and preserves the PR-file-scoped typos workflow fix. - Consolidates the .NET Aspire, AWS SDK, ReportGenerator, JavaScript, and SQLite security updates. - Regenerates the npm, Yarn, and uv lockfiles from the combined manifests instead of resolving generated-file conflicts by selection. - Rolls overlapping updates forward to one compatible result: pyasn1 0.6.4 supersedes 0.6.3; pyOpenSSL 26.2.0 remains compatible with cryptography 48.0.1; and Vite 8/Rolldown makes the older Rollup-only overrides obsolete. - Retains updates already present on current `main` without reintroducing stale pins, including Aspire Hosting Azure Search 13.3.0 and Prompty.Core 2.0.0-beta.3. The full .NET solution builds in Debug with zero warnings and errors. Focused validation passed 82 .NET tests (3 skipped), 1,096 Python tests, both frontend production builds, and uv lock consistency checks for the main Python project and OAuth sample. ### Contribution Checklist - [x] The code builds clean without any errors or warnings - [ ] The PR follows the [SK Contribution Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md) and the [pre-submission formatting script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts) raises no violations - [ ] All unit tests pass, and I have added new tests where possible - [x] I didn't break anyone 😄 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
|
Superseded by #14176 |
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
Bumps flatted from 3.3.3 to 3.4.2.
Commits
3bf09093.4.2885ddccfix CWE-13210bdba70added flatted-view to the benchmark2a02dce3.4.1fba4e8fMerge pull request #89 from WebReflection/python-fix5fe8648added "when in Rome" also a test for PHP53517adsome minor improvementb3e2a0cFixing recursion issue in Python tooc4b46dbAdd SECURITY.md for security policy and reportingf86d071Create dependabot.yml for version updatesYou can trigger a rebase of this PR by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.