@@ -86,22 +86,19 @@ module.exports = async (extensionId, extensionFiles) => {
8686 ) ;
8787 }
8888
89+ const errors = [ ] ;
8990 const registryUrl = `https://${ registryHost } ` ;
9091 for ( const extensionFile of extensionFiles ) {
9192 const xmlManifest = await readXmlManifest ( extensionFile ) ;
92- if (
93- xmlManifest ?. PackageManifest ?. Metadata [ 0 ] ?. Identity [ 0 ] [ "$" ] ?. Publisher . toLowerCase ( ) !=
94- namespace . toLowerCase ( )
93+ const publisher = xmlManifest ?. PackageManifest ?. Metadata [ 0 ] ?. Identity [ 0 ] [ "$" ] ?. Publisher ;
94+ if ( publisher . toLowerCase ( ) != namespace . toLowerCase ( )
9595 ) {
96- console . error (
97- `Namespace name mismatch. Expected ${ namespace } , but found ${ xmlManifest ?. PackageManifest ?. Metadata [ 0 ] ?. Identity [ 0 ] [ "$" ] ?. Publisher } ` ,
98- ) ;
96+ errors . push ( `Namespace name mismatch. Expected ${ namespace } , but found ${ publisher } ` ) ;
9997 continue ;
10098 }
101- if ( xmlManifest ?. PackageManifest ?. Metadata [ 0 ] ?. Identity [ 0 ] [ "$" ] ?. Id . toLowerCase ( ) != extension . toLowerCase ( ) ) {
102- console . error (
103- `Extension name mismatch. Expected ${ extension } , but found ${ xmlManifest ?. PackageManifest ?. Metadata [ 0 ] ?. Identity [ 0 ] [ "$" ] ?. Id } ` ,
104- ) ;
99+ const extensionName = xmlManifest ?. PackageManifest ?. Metadata [ 0 ] ?. Identity [ 0 ] [ "$" ] ?. Id ;
100+ if ( extensionName . toLowerCase ( ) != extension . toLowerCase ( ) ) {
101+ errors . push ( `Extension name mismatch. Expected ${ extension } , but found ${ extensionName } ` ) ;
105102 continue ;
106103 }
107104
@@ -118,4 +115,8 @@ module.exports = async (extensionId, extensionFiles) => {
118115 await ovsx . publish ( options ) ;
119116 console . log ( `Published ${ options . extensionFile } to ${ options . registryUrl } /extension/${ namespace } /${ extension } ` ) ;
120117 }
118+
119+ if ( errors . length > 0 ) {
120+ throw new Error ( errors . join ( '\n' ) )
121+ }
121122} ;
0 commit comments