Skip to content

Commit

Permalink
validate publisher on package
Browse files Browse the repository at this point in the history
  • Loading branch information
benibenj committed Jul 4, 2024
1 parent b6ccc05 commit 67f7374
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions src/package.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ import {
validateVersion,
validateEngineCompatibility,
validateVSCodeTypesCompatibility,
validatePublisher,
} from './validation';
import { detectYarn, getDependencies } from './npm';
import * as GitHost from 'hosted-git-info';
Expand Down Expand Up @@ -1250,6 +1251,7 @@ export class ValidationProcessor extends BaseProcessor {

export function validateManifest(manifest: Manifest): Manifest {
validateExtensionName(manifest.name);
validatePublisher(manifest.publisher);

if (!manifest.version) {
throw new Error('Manifest missing field: version');
Expand Down
2 changes: 1 addition & 1 deletion src/validation.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ export function validatePublisher(publisher: string): void {

if (!nameRegex.test(publisher)) {
throw new Error(
`Invalid publisher name '${publisher}'. Expected the identifier of a publisher, not its human-friendly name. Learn more: https://code.visualstudio.com/api/working-with-extensions/publishing-extension#publishing-extensions`
`Invalid publisher name '${publisher}'. Expected the identifier of a publisher, not its human-friendly name. Learn more: https://code.visualstudio.com/api/working-with-extensions/publishing-extension#publishing-extensions`
);
}
}
Expand Down

0 comments on commit 67f7374

Please sign in to comment.