refactor(config): keep only uid and organizationUid in .cs-launch.json

No resolution spec ever addressed `name`, so `ProjectConfigStore.load()` no
longer keeps it and `projects:create` no longer writes it. The linked-folder
refusal names the project by uid alone.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: SakshiKoli-CS

.talismanrc

@@ -109,4 +109,20 @@ fileignoreconfig:
     - "what `limit` resolves to when nobody passed it"
     - "a test passes `var`, `env-file` and `from-stack`"
     - "an in-flight secret registry to redact against"
+- filename: src/core/project-config.ts
+  allowed_patterns:
+    - "ProjectConfigKey"
+    - "DEFAULT_BLOCK_KEY"
+- filename: src/core/project-config.test.ts
+  allowed_patterns:
+    - "DEFAULT_BLOCK_KEY"
+- filename: src/core/resolve.test.ts
+  allowed_patterns:
+    - "Pass --org, set it in"
+    - "config key and no prompt"
+    - "an explicitly passed --skip 0"
+    - "when the flag was not passed"
+    - "Object\\.keys\\(resolved\\)"
+    - "when every rule passes"
+    - "Pass exactly one of"
 version: "1.0"

AGENTS.md

@@ -417,8 +417,8 @@ that way, keeping the field name; without it every validation 400 read "request
 **The `.cs-launch.json` file.** `ProjectConfigStore` owns it. `load()` returns a typed
 `ProjectConfig`, applying the v1 rule that several branch blocks are usable only when they
 agree on one project. A resolution spec addresses it by a key of `ProjectConfig`, never a
-dotted string. `load()` keeps only the three keys a resolution spec can read - `uid`,
-`organizationUid` and `name` - and only when each is a string or `null`, so a number in the file is
+dotted string. `load()` keeps only the two keys a resolution spec can read - `uid`
+and `organizationUid` - and only when each is a string or `null`, so a number in the file is
 absent rather than a value that reaches a request header. `save()` has one caller, `projects:create`,
 which records the project it created. It merges into every branch block, keeping every other key it
 finds there; it writes a single `project` block into a missing file or one holding an empty object; it refuses, as a

src/core/project-config.test.ts

@@ -281,9 +281,9 @@ describe('ProjectConfigStore.save', () => {
   it('merges into the sole existing block rather than replacing it', () => {
     const store = storeFor({ main: { uid: 'p1', organizationUid: 'org1', name: 'old-name' } });
 
-    store.save({ name: 'new-name' });
+    store.save({ organizationUid: 'org2' });
 
-    expect(fileAt(store)).toEqual({ main: { uid: 'p1', organizationUid: 'org1', name: 'new-name' } });
+    expect(fileAt(store)).toEqual({ main: { uid: 'p1', organizationUid: 'org2', name: 'old-name' } });
   });
 
   it('updates every branch block and discards none of them', () => {
@@ -302,14 +302,14 @@ describe('ProjectConfigStore.save', () => {
       },
     });
 
-    store.save({ uid: 'p1', name: 'renamed' });
+    store.save({ uid: 'p1', organizationUid: 'org2' });
 
     expect(fileAt(store)).toEqual({
-      main: { uid: 'p1', organizationUid: 'org1', name: 'renamed', environments: [{ uid: 'e1', name: 'Default' }] },
+      main: { uid: 'p1', organizationUid: 'org2', name: 'my-site', environments: [{ uid: 'e1', name: 'Default' }] },
       'feature/checkout': {
         uid: 'p1',
-        organizationUid: 'org1',
-        name: 'renamed',
+        organizationUid: 'org2',
+        name: 'my-site',
         environments: [{ uid: 'e2', name: 'Preview' }],
       },
     });
@@ -321,8 +321,8 @@ describe('ProjectConfigStore.save', () => {
       staging: { uid: 'p2', organizationUid: 'org1' },
     });
 
-    expect(() => store.save({ name: 'renamed' })).toThrow(UsageError);
-    expect(() => store.save({ name: 'renamed' })).toThrow('main, staging');
+    expect(() => store.save({ organizationUid: 'org2' })).toThrow(UsageError);
+    expect(() => store.save({ organizationUid: 'org2' })).toThrow('main, staging');
     expect(fileAt(store)).toEqual({
       main: { uid: 'p1', organizationUid: 'org1' },
       staging: { uid: 'p2', organizationUid: 'org1' },
@@ -332,8 +332,8 @@ describe('ProjectConfigStore.save', () => {
   it('refuses to write when the sole existing block names a different project', () => {
     const store = storeFor({ main: { uid: 'p1', organizationUid: 'org1', name: 'other-site' } });
 
-    expect(() => store.save({ uid: 'p2', organizationUid: 'org1', name: 'my-site' })).toThrow(UsageError);
-    expect(() => store.save({ uid: 'p2', organizationUid: 'org1', name: 'my-site' })).toThrow(
+    expect(() => store.save({ uid: 'p2', organizationUid: 'org1' })).toThrow(UsageError);
+    expect(() => store.save({ uid: 'p2', organizationUid: 'org1' })).toThrow(
       `The config file at '${store.path}' already names project p1. Delete it or pass --config with another path.`,
     );
     expect(fileAt(store)).toEqual({ main: { uid: 'p1', organizationUid: 'org1', name: 'other-site' } });
@@ -370,7 +370,7 @@ describe('ProjectConfigStore.save', () => {
 
   it('writes the file so that a later load reads back exactly what was saved', () => {
     const store = new ProjectConfigStore(tempPath());
-    const config = { uid: 'p1', organizationUid: 'org1', name: 'my-site' };
+    const config = { uid: 'p1', organizationUid: 'org1' };
 
     store.save(config);
 
@@ -389,11 +389,11 @@ describe('ProjectConfigStore.save', () => {
     const store = new ProjectConfigStore(path);
 
     const loaded = store.load();
-    store.save({ name: 'renamed' });
+    store.save({ organizationUid: 'org1' });
 
-    expect(loaded).toEqual({ organizationUid: null, name: 'my-site' });
+    expect(loaded).toEqual({ organizationUid: null });
     expect(JSON.parse(readFileSync(path, 'utf8'))).toEqual({
-      project: { uid: 42, organizationUid: null, name: 'renamed', environments: [{ uid: 'e1' }], extra: true },
+      project: { uid: 42, organizationUid: 'org1', name: 'my-site', environments: [{ uid: 'e1' }], extra: true },
     });
   });
 

src/core/project-config.ts

@@ -5,7 +5,6 @@ import { UsageError } from './errors';
 export interface ProjectConfig {
   uid?: string | null;
   organizationUid?: string | null;
-  name?: string | null;
 }
 
 export type ProjectConfigKey = keyof ProjectConfig;
@@ -41,7 +40,7 @@ function textOrNull(value: unknown): string | null | undefined {
 function configFrom(block: Record<string, unknown>): ProjectConfig {
   const config: ProjectConfig = {};
 
-  for (const field of ['uid', 'organizationUid', 'name'] as const) {
+  for (const field of ['uid', 'organizationUid'] as const) {
     const value = textOrNull(block[field]);
 
     if (value !== undefined) {

src/core/resolve.test.ts

@@ -114,7 +114,7 @@ describe('resolveInputs', () => {
   it('advises only the remedies that exist for a required input with no config key and no prompt', async () => {
     const promise = resolveInputs(inputs({ name: { required: true } }), {
       parsed: {},
-      projectConfig: { name: 'from-config' },
+      projectConfig: { organizationUid: 'from-config' },
       services: services({ isTTY: true }),
     });
 

src/projects/project.create.test.ts

@@ -1077,7 +1077,7 @@ describe('ProjectCreator writing the project config', () => {
     await creator.create(gitRequest({ configPath: configPathIn(dataDir) }));
 
     expect(configFileIn(dataDir)).toEqual({
-      project: { uid: PROJECT_UID, organizationUid: ORG, name: 'My Site' },
+      project: { uid: PROJECT_UID, organizationUid: ORG },
     });
   });
 
@@ -1087,7 +1087,7 @@ describe('ProjectCreator writing the project config', () => {
     await creator.create(uploadRequest({ configPath: configPathIn(dataDir) }));
 
     expect(configFileIn(dataDir)).toEqual({
-      project: { uid: PROJECT_UID, organizationUid: ORG, name: 'My Site' },
+      project: { uid: PROJECT_UID, organizationUid: ORG },
     });
   });
 
@@ -1107,43 +1107,32 @@ describe('ProjectCreator writing the project config', () => {
       main: {
         uid: PROJECT_UID,
         organizationUid: ORG,
-        name: 'My Site',
         environments: [{ uid: 'e1', name: 'Default' }],
       },
       'feature/checkout': {
         uid: PROJECT_UID,
         organizationUid: ORG,
-        name: 'My Site',
         environments: [{ uid: 'e2', name: 'Preview' }],
       },
     });
   });
 
   it('refuses before creating anything, and leaves the file alone, when the folder already names a project', async () => {
-    const existing = { project: { uid: 'other-project', organizationUid: ORG, name: 'Other Site' } };
+    const existing = { project: { uid: 'other-project', organizationUid: ORG } };
     writeFileSync(configPathIn(dataDir), JSON.stringify(existing));
     const { creator, created } = harness();
 
     const failure = await creator.create(gitRequest({ configPath: configPathIn(dataDir) })).catch((error: Error) => error);
 
     expect(failure).toBeInstanceOf(UsageError);
     expect((failure as UsageError).message).toBe(
-      `This folder is already linked to the project "Other Site" (other-project) in ${configPathIn(dataDir)}. ` +
+      `This folder is already linked to the project other-project in ${configPathIn(dataDir)}. ` +
         'To create a new project, remove that file or pass --config with a different path.',
     );
     expect(created).toEqual([]);
     expect(configFileIn(dataDir)).toEqual(existing);
   });
 
-  it('names a linked project by uid alone when the file carries no name', async () => {
-    writeFileSync(configPathIn(dataDir), JSON.stringify({ project: { uid: 'other-project' } }));
-    const { creator } = harness();
-
-    const failure = await creator.create(gitRequest({ configPath: configPathIn(dataDir) })).catch((error: Error) => error);
-
-    expect((failure as UsageError).message).toContain('already linked to the project other-project in');
-  });
-
   it('reports the miss and leaves an unparseable config file untouched, still completing the create', async () => {
     writeFileSync(configPathIn(dataDir), '{ not json ');
     const { creator, printed, created } = harness();

src/projects/project.create.ts

@@ -183,10 +183,6 @@ export class ProjectCreator {
 
     const config: ProjectConfig = { uid: project.uid, organizationUid: request.org };
 
-    if (project.name !== undefined) {
-      config.name = project.name;
-    }
-
     try {
       new ProjectConfigStore(path).save(config);
     } catch (error) {
@@ -494,10 +490,8 @@ export class ProjectCreator {
       return;
     }
 
-    const named = linked.name ? `"${linked.name}" (${linked.uid})` : `${linked.uid}`;
-
     throw new UsageError(
-      `This folder is already linked to the project ${named} in ${request.configPath}. ` +
+      `This folder is already linked to the project ${linked.uid} in ${request.configPath}. ` +
         'To create a new project, remove that file or pass --config with a different path.',
     );
   }

test/integration/projects-create.test.ts

@@ -892,7 +892,7 @@ describe('integration: launch:projects:create on the wire', () => {
     const other = randomBytes(12).toString('hex');
     writeFileSync(
       join(dataDir, '.cs-launch.json'),
-      JSON.stringify({ project: { uid: other, organizationUid: ORG_UID, name: 'Existing Site' } }),
+      JSON.stringify({ project: { uid: other, organizationUid: ORG_UID } }),
     );
     const signed = hub().get('/manage/projects/upload/signed_url').query({}).reply(200, awsSignedUpload());
     const create = captureCreate();
@@ -917,7 +917,7 @@ describe('integration: launch:projects:create on the wire', () => {
     );
 
     expect(error?.oclif?.exit).toBe(2);
-    expect(error?.message).toContain(`already linked to the project "Existing Site" (${other})`);
+    expect(error?.message).toContain(`already linked to the project ${other}`);
     expect(error?.message).toContain('--config');
     expect(signed.isDone()).toBe(false);
     expect(create.scope.isDone()).toBe(false);