chore: remove legacy code related to w3c-common profile (#2839)

Author: sidvishnoi

src/core/data-cite.js

@@ -10,13 +10,13 @@
  * directly by the author as well as via other modules like core/xref.
  */
 import { biblio, resolveRef, updateFromNetwork } from "./biblio.js";
-import { pub, sub } from "./pubsubhub.js";
 import {
   refTypeFromContext,
   showInlineError,
   showInlineWarning,
   wrapInner,
 } from "./utils.js";
+import { sub } from "./pubsubhub.js";
 export const name = "core/data-cite";
 
 /**
@@ -178,13 +178,6 @@ export async function run() {
   }
 
   sub("beforesave", cleanup);
-
-  // Added message for legacy compat with Aria specs
-  // See https://github.com/w3c/respec/issues/793,
-  //
-  // Why `core/link-to-dfn` and not `core/data-cite`? For backward compatibility
-  // after a refactor (https://github.com/w3c/respec/issues/2830)
-  pub("end", "core/link-to-dfn");
 }
 
 /**

src/core/location-hash.js

@@ -2,13 +2,9 @@
 // Module core/location-hash
 // Resets window.location.hash to jump to the right point in the document
 
-import { pub } from "./pubsubhub.js";
 export const name = "core/location-hash";
 
 export function run() {
-  // Added message for legacy compat with Aria specs
-  // See https://github.com/w3c/respec/issues/793
-  pub("start", "core/location-hash");
   if (!location.hash) {
     return;
   }

src/core/override-configuration.js

@@ -10,9 +10,7 @@ import { pub, sub } from "./pubsubhub.js";
 export const name = "core/override-configuration";
 
 function overrideConfig(config) {
-  // For legacy reasons, we still support both ";" and "&"
-  const searchQuery = document.location.search.replace(/;/g, "&");
-  const params = new URLSearchParams(searchQuery);
+  const params = new URLSearchParams(document.location.search);
   const overrideEntries = Array.from(params)
     .filter(([key, value]) => !!key && !!value)
     .map(([codedKey, codedValue]) => {

src/dini/conformance.js

@@ -89,7 +89,4 @@ export function run(conf) {
         'Please add a `<section id="conformance">`.'
     );
   }
-  // Added message for legacy compat with Aria specs
-  // See https://github.com/w3c/respec/issues/793
-  pub("end", name);
 }

src/dini/defaults.js

@@ -4,7 +4,6 @@
  */
 export const name = "dini/defaults";
 import { coreDefaults } from "../core/defaults.js";
-import { definitionMap } from "../core/dfn-map.js";
 import linter from "../core/linter.js";
 import { rule as privsecSectionRule } from "../core/linter-rules/privsec-section.js";
 
@@ -80,9 +79,4 @@ export function run(conf) {
 
   // computed properties
   Object.assign(conf, computeProps(conf));
-
-  // TODO: eventually, we want to remove this.
-  // It's here for legacy support of json-ld specs
-  // see https://github.com/w3c/respec/issues/2019
-  Object.assign(conf, { ...Object.fromEntries(definitionMap) });
 }

src/w3c/conformance.js

@@ -89,7 +89,4 @@ export function run(conf) {
         'Please add a `<section id="conformance">`.'
     );
   }
-  // Added message for legacy compat with Aria specs
-  // See https://github.com/w3c/respec/issues/793
-  pub("end", name);
 }

src/w3c/defaults.js

@@ -4,7 +4,6 @@
  */
 export const name = "w3c/defaults";
 import { coreDefaults } from "../core/defaults.js";
-import { definitionMap } from "../core/dfn-map.js";
 import linter from "../core/linter.js";
 import { rule as privsecSectionRule } from "../core/linter-rules/privsec-section.js";
 import { rule as wptTestsExist } from "../core/linter-rules/wpt-tests-exist.js";
@@ -46,9 +45,4 @@ export function run(conf) {
     ...conf,
     lint,
   });
-
-  // TODO: eventually, we want to remove this.
-  // It's here for legacy support of json-ld specs
-  // see https://github.com/w3c/respec/issues/2019
-  Object.assign(conf, { ...Object.fromEntries(definitionMap) });
 }