PetschStudios/KanbanGit
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases 27 Wiki Activity

codex/public-api-docs #38

Merged
Twyco merged 6 commits from codex/public-api-docs into main 2026-04-24 09:11:49 +00:00
Conversation 4 Commits 6 Files Changed 5
+6
2 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit a6a4615ad5 - Show all commits
config/scalar.php
+5
View File
@@ -112,6 +112,11 @@ return [
*/ */
'hideTestRequestButton' => true, 'hideTestRequestButton' => true,
/**
* Whether to show the API client button in the sidebar and modal.
*/
'hideClientButton' => true,
/** /**
* MCP will be introduced as a first-party KanbanGit feature later. * MCP will be introduced as a first-party KanbanGit feature later.
*/ */
tests/Feature/Api/PublicApiDocumentationTest.php
+1
View File
@@ -10,6 +10,7 @@ class PublicApiDocumentationTest extends TestCase
{ {
$this->get('/api/docs') $this->get('/api/docs')
->assertOk() ->assertOk()
->assertSee('"hideClientButton":true', false)
Koda commented 2026-04-24 09:03:13 +00:00
Outdated
Review
Copy Link
Copy Source

Nit/Optional: These assertions depend on Scalar's exact minified JSON serialization ("hideClientButton":true / "mcp":{"disabled":true}), which is brittle across package updates. Prefer asserting stable behavior (e.g., response OK plus the configured OpenAPI URL) instead of exact formatting.

Nit/Optional: These assertions depend on Scalar's exact minified JSON serialization (`"hideClientButton":true` / `"mcp":{"disabled":true}`), which is brittle across package updates. Prefer asserting stable behavior (e.g., response OK plus the configured OpenAPI URL) instead of exact formatting.
->assertSee('"mcp":{"disabled":true}', false); ->assertSee('"mcp":{"disabled":true}', false);
} }