fix(core): update Manifest.raw when persisting a Workspace (#2726)

* fix: reload raw manifest when persisting the manifest

* test: check manifest.raw is updated

Co-authored-by: merceyz <[email protected]>

Author: noahnu

.yarn/versions/bb9f6051.yml

@@ -0,0 +1,30 @@
+releases:
+  "@yarnpkg/cli": patch
+  "@yarnpkg/core": patch
+
+declined:
+  - "@yarnpkg/plugin-compat"
+  - "@yarnpkg/plugin-constraints"
+  - "@yarnpkg/plugin-dlx"
+  - "@yarnpkg/plugin-essentials"
+  - "@yarnpkg/plugin-exec"
+  - "@yarnpkg/plugin-file"
+  - "@yarnpkg/plugin-git"
+  - "@yarnpkg/plugin-github"
+  - "@yarnpkg/plugin-http"
+  - "@yarnpkg/plugin-init"
+  - "@yarnpkg/plugin-interactive-tools"
+  - "@yarnpkg/plugin-link"
+  - "@yarnpkg/plugin-node-modules"
+  - "@yarnpkg/plugin-npm"
+  - "@yarnpkg/plugin-npm-cli"
+  - "@yarnpkg/plugin-pack"
+  - "@yarnpkg/plugin-patch"
+  - "@yarnpkg/plugin-pnp"
+  - "@yarnpkg/plugin-stage"
+  - "@yarnpkg/plugin-typescript"
+  - "@yarnpkg/plugin-version"
+  - "@yarnpkg/plugin-workspace-tools"
+  - "@yarnpkg/builder"
+  - "@yarnpkg/doctor"
+  - "@yarnpkg/pnpify"

packages/yarnpkg-core/sources/Workspace.ts

@@ -189,5 +189,7 @@ export class Workspace {
     await xfs.changeFilePromise(path, content, {
       automaticNewlines: true,
     });
+
+    this.manifest.raw = data;
   }
 }

packages/yarnpkg-core/tests/Project.test.ts

@@ -138,4 +138,20 @@ describe(`Project`, () => {
       });
     });
   });
+
+  it(`should update Manifest.raw when persisting a workspace`, async () => {
+    await xfs.mktempPromise(async path => {
+      await xfs.writeJsonPromise(ppath.join(path, Filename.manifest), {name: `foo`});
+      await xfs.writeFilePromise(ppath.join(path, Filename.lockfile), ``);
+
+      const configuration = getConfiguration(path);
+      const {project} = await Project.find(configuration, path);
+
+      expect(project.topLevelWorkspace.manifest.raw.main).toBeUndefined();
+
+      project.topLevelWorkspace.manifest.main = `./index.js` as PortablePath;
+      await project.topLevelWorkspace.persistManifest();
+      expect(project.topLevelWorkspace.manifest.raw.main).toEqual(`./index.js`);
+    });
+  });
 });