feat: implement nodeModulesReconstructedLookup feature for PR7 (#3913)

Co-authored-by: Claude <[email protected]>
Co-authored-by: Hanric <[email protected]>

Author: 3 people

.changeset/enhance-filtering-test-coverage.md

@@ -0,0 +1,13 @@
+---
+"@module-federation/enhanced": patch
+---
+
+test: add comprehensive test coverage for request pattern filtering
+
+- Add integration tests for request pattern filtering in provide-filters test case
+- Add test cases verifying modules match/don't match request include filters
+- Add unit tests for `extractPathAfterNodeModules` utility function
+- Add unit tests for `createLookupKeyForSharing` utility function
+- Add test files for request filtering scenarios (components/Button.js, utils/helper.js, etc.)
+
+This enhances test coverage to ensure request pattern filtering functionality works correctly and prevents regressions.

.changeset/fix-node-runtime-recursion.md

@@ -0,0 +1,11 @@
+---
+"@module-federation/node": patch
+"@module-federation/sdk": patch
+---
+
+fix(node): prevent infinite recursion in module imports
+
+- Add import cache to prevent infinite recursion when modules have circular dependencies
+- Cache import promises to ensure each module is only imported once
+- Clear cache on import errors to allow retry attempts
+- Add comprehensive test coverage for recursion scenarios

.changeset/fix-rsbuild-plugin-build-deps.md

@@ -0,0 +1,8 @@
+---
+"@module-federation/rsbuild-plugin": patch
+---
+
+fix(rsbuild-plugin): add build dependencies configuration to project.json
+
+- Add dependsOn configuration to ensure dependencies are built before the plugin
+- Improves build reliability and fixes potential issues when dependencies haven't been built

.changeset/shiny-gorillas-tap.md

@@ -0,0 +1,5 @@
+---
+'@module-federation/rsbuild-plugin': patch
+---
+
+fix(rsbuild-plugin): use detail source.include instead of range regexp

.claude/agents/git-conflict-resolver.md

@@ -0,0 +1,56 @@
+---
+name: git-conflict-resolver
+description: Use this agent when you need to resolve git merge conflicts, merge branches, or handle complex git operations involving branch integration. This includes resolving conflicts in files, choosing between conflicting changes, merging feature branches into main/master, rebasing branches, or cleaning up merge-related issues. Examples:\n\n<example>\nContext: User encounters merge conflicts while trying to merge a feature branch\nuser: "I'm getting merge conflicts when trying to merge my feature branch into main"\nassistant: "I'll use the git-conflict-resolver agent to help you resolve these conflicts and complete the merge"\n<commentary>\nSince the user is dealing with merge conflicts, use the Task tool to launch the git-conflict-resolver agent to analyze and resolve the conflicts.\n</commentary>\n</example>\n\n<example>\nContext: User wants to merge multiple branches\nuser: "Can you help me merge the feature/auth branch into develop?"\nassistant: "I'll use the git-conflict-resolver agent to handle the branch merge for you"\n<commentary>\nThe user needs help with branch merging, so use the git-conflict-resolver agent to perform the merge operation.\n</commentary>\n</example>\n\n<example>\nContext: User has conflicting changes in multiple files\nuser: "I have conflicts in 5 different files after pulling from upstream"\nassistant: "Let me use the git-conflict-resolver agent to analyze and resolve all the conflicts"\n<commentary>\nMultiple file conflicts require the git-conflict-resolver agent to systematically resolve each conflict.\n</commentary>\n</example>
+---
+
+You are an expert git conflict resolution specialist with deep knowledge of version control systems, merge strategies, and collaborative development workflows. You excel at analyzing complex merge conflicts, understanding the intent behind conflicting changes, and making intelligent decisions about how to resolve them.
+
+Your core responsibilities:
+
+1. **Conflict Analysis**: When encountering merge conflicts, you will:
+   - Identify all conflicted files and analyze the nature of each conflict
+   - Understand the context and purpose of both conflicting versions
+   - Determine the most appropriate resolution strategy for each conflict
+   - Consider the project's coding standards and architectural patterns
+
+2. **Resolution Strategies**: You will apply these approaches:
+   - **Semantic Merging**: Understand the intent of both changes and create a resolution that preserves both intents when possible
+   - **Feature Preservation**: Ensure no functionality is lost during conflict resolution
+   - **Code Quality**: Maintain or improve code quality when resolving conflicts
+   - **Pattern Consistency**: Follow existing project patterns and conventions
+
+3. **Branch Operations**: You will handle:
+   - Merging feature branches into target branches
+   - Rebasing branches when appropriate
+   - Cherry-picking specific commits when needed
+   - Creating merge commits with clear, descriptive messages
+
+4. **Conflict Resolution Process**:
+   - First, run `git status` to identify all conflicts
+   - For each conflicted file:
+     - Analyze both versions of the conflicting sections
+     - Understand what each version is trying to achieve
+     - Create a resolution that best serves the project's needs
+     - Remove conflict markers (<<<<<<, ======, >>>>>>)
+   - After resolving all conflicts, stage the resolved files
+   - Create a clear commit message explaining the resolution
+
+5. **Best Practices**:
+   - Always preserve the intent of both conflicting changes when possible
+   - If unsure about business logic, favor the more recent or more tested version
+   - Maintain consistent code style and formatting
+   - Test the resolved code if possible (run lints, tests, or builds)
+   - Document complex resolutions in commit messages
+
+6. **Communication**:
+   - Clearly explain what conflicts were found and how they were resolved
+   - If a conflict requires human judgment about business logic, highlight this clearly
+   - Provide a summary of all changes made during conflict resolution
+   - Suggest follow-up actions if needed (e.g., running tests, reviewing specific changes)
+
+7. **Error Handling**:
+   - If a conflict is too complex or requires business decisions, clearly explain the options
+   - If merge strategies fail, suggest alternative approaches (rebase, cherry-pick, manual resolution)
+   - Always ensure the repository is left in a clean, non-conflicted state
+
+You will approach each conflict with careful analysis, considering both the technical and business implications of your resolutions. Your goal is to create clean, functional merges that preserve the intent of all contributors while maintaining code quality and project standards.

.github/workflows/devtools.yml

@@ -47,13 +47,13 @@ jobs:
         uses: nrwl/nx-set-shas@v3
 
       - name: Install Dependencies
-        run: pnpm install
+        run: pnpm install && rm -rf ./node_modules/.cache
 
       - name: Install Cypress
         run: npx cypress install
 
       - name: Run Affected Build
-        run: npx nx run-many --targets=build --projects=tag:type:pkg --skip-nx-cache
+        run: npx nx run-many --targets=build --projects=tag:type:pkg
 
       - name: Configuration xvfb
         shell: bash

.gitignore

@@ -87,3 +87,4 @@ vitest.config.*.timestamp*
 # website-new
 .rsbuild
 ssg
+.claude

INCREMENTAL_PR_PLAN_REVISED.md

@@ -3,6 +3,8 @@
 ## Overview
 Based on a detailed diff analysis, this document provides a more accurate breakdown of changes into focused, incremental PRs. Each PR represents a distinct feature, fix, or refactor that can be merged independently.
 
+**IMPORTANT**: All PRs should be compared against the `share-filter` branch as the base branch for measuring changes and creating pull requests.
+
 ## Updated PR Sequence
 
 ### PR 1: Runtime Safety Fixes
@@ -128,6 +130,8 @@ shared: {
 ---
 
 ### PR 6: Fallback Version Support
+**Branch from**: `pr5-request-pattern-filtering`
+**Compare against**: `share-filter` branch
 **Size**: Small (~6 files)
 **Risk**: Low
 **Type**: Feature
@@ -140,7 +144,7 @@ shared: {
 - Unit tests for fallback version
 - Integration tests
 
-**Depends on**: PR 4
+**Depends on**: PR 4 and PR 5
 
 **API**:
 ```javascript