Skip to content

Commit a36c9ad

Browse files
authored
Merge pull request #227 from abraham/copilot/fix-226
Set default value 'read' for createApp scopes property
2 parents ba63086 + c9784bb commit a36c9ad

File tree

3 files changed

+6
-1
lines changed

3 files changed

+6
-1
lines changed

dist/schema.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5155,7 +5155,8 @@
51555155
"scopes": {
51565156
"type": "string",
51575157
"format": "scopes",
5158-
"description": "Space separated list of scopes. If none is provided, defaults to `read`. See [OAuth Scopes] for a list of possible scopes."
5158+
"description": "Space separated list of scopes. If none is provided, defaults to `read`. See [OAuth Scopes] for a list of possible scopes.",
5159+
"default": "read"
51595160
},
51605161
"website": {
51615162
"type": "string",

src/__tests__/integration/create-app-scopes-override.test.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,5 +39,8 @@ describe('CreateApp scopes override', () => {
3939

4040
// Verify the description is preserved
4141
expect(scopesProperty.description).toContain('scopes');
42+
43+
// Verify the default value is set to 'read'
44+
expect(scopesProperty.default).toBe('read');
4245
});
4346
});

src/generators/MethodConverter.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -304,6 +304,7 @@ class MethodConverter {
304304
type: 'string',
305305
format: 'scopes',
306306
description: properties.scopes.description,
307+
default: 'read',
307308
};
308309
}
309310

0 commit comments

Comments
 (0)