Skip to content

Commit 93a0e14

Browse files
committed
rename to make
1 parent 0746896 commit 93a0e14

File tree

4 files changed

+20
-20
lines changed

4 files changed

+20
-20
lines changed

src/Prompts/Contracts/PromptFactory.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,5 @@
66

77
interface PromptFactory
88
{
9-
public function create(string $id): PromptTemplate;
9+
public function make(string $name): PromptTemplate;
1010
}

src/Prompts/Factories/LangfusePromptFactory.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,9 @@ public function __construct(
7575
protected ?Closure $metadataResolver = null,
7676
) {}
7777

78-
public function create(string $id): PromptTemplate
78+
public function make(string $name): PromptTemplate
7979
{
80-
$data = $this->getResponseContent($id);
80+
$data = $this->getResponseContent($name);
8181

8282
$builder = match ($data['type']) {
8383
'chat' => $this->buildChatPrompt($data),
@@ -137,7 +137,7 @@ protected function buildTextPrompt(array $data): TextPromptBuilder
137137
/**
138138
* @return LangfusePromptResponse
139139
*/
140-
protected function getResponseContent(string $id): array
140+
protected function getResponseContent(string $name): array
141141
{
142142
$params = [];
143143

@@ -149,7 +149,7 @@ protected function getResponseContent(string $id): array
149149
$params['label'] = $this->label;
150150
}
151151

152-
$uri = sprintf('%s/api/public/v2/prompts/%s', $this->baseUrl, $id);
152+
$uri = sprintf('%s/api/public/v2/prompts/%s', $this->baseUrl, $name);
153153

154154
if ($params !== []) {
155155
$uri .= '?' . http_build_query($params);

src/Prompts/Factories/McpPromptFactory.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
*/
1414
class McpPromptFactory implements PromptFactory
1515
{
16-
public function create(string $id): PromptTemplate
16+
public function make(string $name): PromptTemplate
1717
{
1818
// TODO: Implement
1919
return new ChatPromptTemplate([]);

tests/Unit/Prompts/Factories/LangfusePromptFactoryTest.php

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ function createLangfuseFactory(
6262
new Response(200, body: json_encode($responseData)),
6363
]);
6464

65-
$prompt = $factory->create('test-chat-prompt');
65+
$prompt = $factory->make('test-chat-prompt');
6666

6767
expect($prompt)->toBeInstanceOf(ChatPromptTemplate::class);
6868
expect($prompt->messages)->toHaveCount(3);
@@ -89,7 +89,7 @@ function createLangfuseFactory(
8989
new Response(200, body: json_encode($responseData)),
9090
]);
9191

92-
$prompt = $factory->create('test-text-prompt');
92+
$prompt = $factory->make('test-text-prompt');
9393

9494
expect($prompt)->toBeInstanceOf(TextPromptTemplate::class);
9595
expect($prompt->text)->toBe('Write a story about {{topic}}. Make it {{length}} words long.');
@@ -121,7 +121,7 @@ function createLangfuseFactory(
121121
new Response(200, body: json_encode($responseData)),
122122
]);
123123

124-
$prompt = $factory->create('test-assistant-prompt');
124+
$prompt = $factory->make('test-assistant-prompt');
125125

126126
expect($prompt->messages[0])->toBeInstanceOf(UserMessage::class);
127127
expect($prompt->messages[1])->toBeInstanceOf(AssistantMessage::class);
@@ -143,7 +143,7 @@ function createLangfuseFactory(
143143
new Response(200, body: json_encode($responseData)),
144144
]);
145145

146-
expect(fn(): PromptTemplate => $factory->create('test-prompt'))
146+
expect(fn(): PromptTemplate => $factory->make('test-prompt'))
147147
->toThrow(PromptException::class, 'Unsupported prompt type: unsupported');
148148
});
149149

@@ -168,7 +168,7 @@ function createLangfuseFactory(
168168
new Response(200, body: json_encode($responseData)),
169169
]);
170170

171-
expect(fn(): PromptTemplate => $factory->create('test-prompt'))
171+
expect(fn(): PromptTemplate => $factory->make('test-prompt'))
172172
->toThrow(PromptException::class, 'Unsupported message role: unknown');
173173
});
174174

@@ -192,7 +192,7 @@ function createLangfuseFactory(
192192
new Response(200, body: json_encode($responseData)),
193193
]);
194194

195-
expect(fn(): PromptTemplate => $factory->create('test-prompt'))
195+
expect(fn(): PromptTemplate => $factory->make('test-prompt'))
196196
->toThrow(PromptException::class, 'Unsupported message type: unknown');
197197
});
198198

@@ -211,7 +211,7 @@ function createLangfuseFactory(
211211
new Response(200, body: json_encode($responseData)),
212212
]);
213213

214-
$prompt = $factory->create('test-prompt');
214+
$prompt = $factory->make('test-prompt');
215215

216216
expect($prompt)->toBeInstanceOf(TextPromptTemplate::class);
217217
});
@@ -221,7 +221,7 @@ function createLangfuseFactory(
221221
new Response(404, body: 'Prompt not found'),
222222
]);
223223

224-
expect(fn(): PromptTemplate => $factory->create('nonexistent-prompt'))
224+
expect(fn(): PromptTemplate => $factory->make('nonexistent-prompt'))
225225
->toThrow(PromptException::class, 'Invalid JSON response from Langfuse: Syntax error');
226226
});
227227

@@ -230,7 +230,7 @@ function createLangfuseFactory(
230230
new Response(200, body: 'invalid json {'),
231231
]);
232232

233-
expect(fn(): PromptTemplate => $factory->create('test-prompt'))
233+
expect(fn(): PromptTemplate => $factory->make('test-prompt'))
234234
->toThrow(PromptException::class, 'Invalid JSON response from Langfuse: Syntax error');
235235
});
236236

@@ -270,7 +270,7 @@ function createLangfuseFactory(
270270
]);
271271

272272
/** @var \Cortex\Prompts\Templates\TextPromptTemplate $prompt */
273-
$prompt = $factory->create('test-prompt');
273+
$prompt = $factory->make('test-prompt');
274274

275275
expect($prompt)->toBeInstanceOf(TextPromptTemplate::class);
276276
expect($prompt->metadata)->not()->toBeNull();
@@ -309,7 +309,7 @@ function createLangfuseFactory(
309309
new Response(200, body: json_encode($responseData)),
310310
]);
311311

312-
$prompt = $factory->create('test-prompt');
312+
$prompt = $factory->make('test-prompt');
313313

314314
expect($prompt)->toBeInstanceOf(ChatPromptTemplate::class);
315315
expect($prompt->metadata)->not()->toBeNull();
@@ -342,7 +342,7 @@ function createLangfuseFactory(
342342
new Response(200, body: json_encode($responseData)),
343343
]);
344344

345-
$prompt = $factory->create('test-prompt');
345+
$prompt = $factory->make('test-prompt');
346346

347347
expect($prompt->metadata)->not()->toBeNull();
348348
expect($prompt->metadata->provider)->toBe('anthropic');
@@ -390,7 +390,7 @@ function createLangfuseFactory(
390390
);
391391
});
392392

393-
$prompt = $factory->create('test-prompt');
393+
$prompt = $factory->make('test-prompt');
394394

395395
expect($prompt->metadata)->not()->toBeNull();
396396
expect($prompt->metadata->provider)->toBe('custom-provider');
@@ -438,7 +438,7 @@ function createLangfuseFactory(
438438
new Response(200, body: json_encode($responseData)),
439439
]);
440440

441-
$prompt = $factory->create('test-prompt');
441+
$prompt = $factory->make('test-prompt');
442442

443443
expect($prompt)->toBeInstanceOf(ChatPromptTemplate::class);
444444
expect($prompt->metadata)->not()->toBeNull();

0 commit comments

Comments
 (0)