fixed ADR file path in UrlResolver

Signed-off-by: Deepankumar Loganathan <deepan0433@gmail.com>

plugins/adr-common/api-report.md

@@ -25,6 +25,7 @@ export const ANNOTATION_ADR_LOCATION = 'backstage.io/adr-location';
 export const getAdrLocationUrl: (
   entity: Entity,
   scmIntegration: ScmIntegrationRegistry,
+  adrFilePath?: String,
 ) => string;
 
 // @public

plugins/adr-common/src/index.ts

@@ -57,11 +57,19 @@ export const isAdrAvailable = (entity: Entity) =>
 export const getAdrLocationUrl = (
   entity: Entity,
   scmIntegration: ScmIntegrationRegistry,
+  adrFilePath?: String,
 ) => {
   if (!isAdrAvailable(entity)) {
     throw new Error(`Missing ADR annotation: ${ANNOTATION_ADR_LOCATION}`);
   }
 
+  if (adrFilePath) {
+    return scmIntegration.resolveUrl({
+      url: `${getAdrLocationDir(entity)!.replace(/\/$/, '')}/${adrFilePath}`,
+      base: getEntitySourceLocation(entity).target,
+    });
+  }
+
   return scmIntegration.resolveUrl({
     url: getAdrLocationDir(entity)!,
     base: getEntitySourceLocation(entity).target,

plugins/adr/src/components/AdrReader/AdrReader.tsx

@@ -44,22 +44,11 @@ export const AdrReader = (props: {
   const { entity } = useEntity();
   const scmIntegrations = useApi(scmIntegrationsApiRef);
   const adrApi = useApi(adrApiRef);
-  const adrLocationUrl = getAdrLocationUrl(entity, scmIntegrations);
-  let url = `${adrLocationUrl.replace(/\/$/, '')}`;
-  const adrUrlPath = url.match(/path=\/.*\&/);
-  if (adrUrlPath) {
-    // Azure DevOps SCM handle the path in URL Params
-    const adrPath = adrUrlPath![0].replace(/\&$/, '');
-    const regex = new RegExp(`${adrPath}`);
-    url = url.replace(regex, `${adrPath}/${adr}}`);
-  } else {
-    // Other SCM tools
-    url = `${url}/${adr}`;
-  }
+  const adrLocationUrl = getAdrLocationUrl(entity, scmIntegrations, adr);
 
   const { value, loading, error } = useAsync(
-    async () => adrApi.readAdr(url),
-    [url],
+    async () => adrApi.readAdr(adrLocationUrl),
+    [adrLocationUrl],
   );
 
   const adrContent = useMemo(() => {