Drop unused code

src/analysis/analysisVisitors/renderersAnalysis.ts

@@ -26,7 +26,7 @@ import { PipelineFlavor } from "@/pageEditor/pageEditorTypes";
 
 export const MULTIPLE_RENDERERS_ERROR_MESSAGE =
   "A panel can only have one renderer. There are one or more other renderers configured for this extension.";
-export const RENDERER_MUST_BE_LAST_BLOCK_ERROR_MESSAGE =
+const RENDERER_MUST_BE_LAST_BLOCK_ERROR_MESSAGE =
   "A renderer must be the last brick.";
 
 class RenderersAnalysis extends AnalysisVisitorWithResolvedBlocks {

src/analysis/analysisVisitors/varAnalysis/varMap.ts

@@ -37,7 +37,7 @@ export type ExistenceNode = {
  * @param selfExistence Existence of the current node
  * @param allowAnyChild Whether the current node allows any child (i.e. doesn't have  a strict schema)
  */
-export function createNode(
+function createNode(
   selfExistence: VarExistence,
   allowAnyChild = false
 ): ExistenceNode {

src/auth/RequireAuth.tsx

@@ -45,7 +45,7 @@ type RequireAuthProps = {
 /**
  * Hook to determine authentication status. Authentication can be via native PixieBrix token, or partner Bearer JWT.
  */
-export const useRequiredAuth = () => {
+const useRequiredAuth = () => {
   const dispatch = useDispatch();
   const hasCachedLoggedIn = useSelector(selectIsLoggedIn);
   const { hasToken, tokenLoading, tokenError } = useLinkState();

src/auth/authUtils.ts

@@ -19,7 +19,7 @@ import { type Me } from "@/types/contract";
 import { type UserDataUpdate, type AuthState } from "@/auth/authTypes";
 import { type UUID } from "@/core";
 
-export function selectOrganizations(
+function selectOrganizations(
   organizationMemberships: Me["organization_memberships"]
 ): AuthState["organizations"] {
   if (organizationMemberships == null) {

src/auth/useRequiredPartnerAuth.ts

@@ -47,7 +47,7 @@ const PARTNER_MAP = new Map<string, Set<RegistryId>>([
   ],
 ]);
 
-export type RequiredPartnerState = {
+type RequiredPartnerState = {
   /**
    * True if the user's is a partner account.
    */

src/background/contextMenus.ts

@@ -41,7 +41,7 @@ const MENU_PREFIX = "pixiebrix-";
 // >= 1 seconds because it also waits for the content script to be ready
 const CONTEXT_SCRIPT_INSTALL_MS = 5000;
 
-export type SelectionMenuOptions = {
+type SelectionMenuOptions = {
   extensionId: UUID;
   title: string;
   contexts: Menus.ContextType[];

src/background/dataStore.ts

@@ -18,8 +18,8 @@
 import { type ManualStorageKey, readStorage, setStorage } from "@/chrome";
 import { type JsonObject } from "type-fest";
 
-export const LOCAL_DATA_STORE = "LOCAL_DATA_STORE" as ManualStorageKey;
-export const KEY_PREFIX = "@@";
+const LOCAL_DATA_STORE = "LOCAL_DATA_STORE" as ManualStorageKey;
+const KEY_PREFIX = "@@";
 
 export async function getRecord(primaryKey: string): Promise<unknown> {
   const data = await readStorage<Record<string, unknown>>(LOCAL_DATA_STORE, {});

src/background/externalProtocol.ts

@@ -35,15 +35,15 @@ import {
 
 type ChromeMessageSender = chrome.runtime.MessageSender;
 
-export const MESSAGE_PREFIX = "@@pixiebrix/background/";
+const MESSAGE_PREFIX = "@@pixiebrix/background/";
 
 const handlers = new Map<string, HandlerEntry>();
 
 /**
  * Return true if a message sender is either the extension itself, or an externally connectable page
  * https://developer.chrome.com/extensions/security#sanitize
  */
-export function allowBackgroundSender(
+function allowBackgroundSender(
   sender: ChromeMessageSender | Runtime.MessageSender
 ): boolean {
   if (sender == null) {

src/background/partnerIntegrations.ts

@@ -33,7 +33,7 @@ import axios from "axios";
 /**
  * A principal on a remote service, e.g., an Automation Anywhere Control Room.
  */
-export type PartnerPrincipal = {
+type PartnerPrincipal = {
   /**
    * The hostname of the remote service, e.g., the Automation Anywhere Control Room.
    */

src/background/requests.ts

@@ -234,9 +234,7 @@ async function proxyRequest<T>(
 
 const UNAUTHORIZED_STATUS_CODES = new Set([401, 403]);
 
-export function isAuthenticationError(
-  error: Pick<AxiosError, "response">
-): boolean {
+function isAuthenticationError(error: Pick<AxiosError, "response">): boolean {
   // Response should be an object, but be defensive
   if (error.response == null || !isObject(error.response)) {
     return false;

src/blocks/PipelineExpressionVisitor.ts

@@ -25,7 +25,7 @@ import PipelineVisitor, {
 } from "./PipelineVisitor";
 import { type DocumentElement } from "@/components/documentBuilder/documentBuilderTypes";
 
-export type VisitDocumentElementArgs = {
+type VisitDocumentElementArgs = {
   position: BlockPosition;
   blockConfig: BlockConfig;
   element: DocumentElement;

src/blocks/PipelineVisitor.ts

@@ -67,7 +67,7 @@ export type VisitPipelineExtra = {
    */
   pipelinePropName?: string | undefined;
 };
-export type VisitRootPipelineExtra = {
+type VisitRootPipelineExtra = {
   extensionPointType: ExtensionPointType;
 };
 

src/blocks/effects/highlightText.ts

@@ -28,7 +28,7 @@ const highlightId = validateRegistryId("@pixiebrix/html/highlight-text");
 /**
  * Recursively wrap text in an element and its children.
  */
-export function wrapText({
+function wrapText({
   nodes,
   pattern,
   color,

src/blocks/effects/replaceText.ts

@@ -46,7 +46,7 @@ export function getTextNodes(roots: Node[]): Text[] {
 /**
  * Recursively replace text in an element and its children, without modifying the structure of the document.
  */
-export function replaceText({
+function replaceText({
   nodes,
   pattern,
   replacement,

src/blocks/readers/factory.ts

@@ -81,17 +81,6 @@ export function registerFactory(readerType: string, read: Read): void {
   _readerFactories.set(readerType, read);
 }
 
-export function makeRead(
-  config: ReaderTypeConfig
-): (root: ReaderRoot) => Promise<ReaderOutput> {
-  const doRead = _readerFactories.get(config.type);
-  if (!doRead) {
-    throw new Error(`Reader type ${config.type} not implemented`);
-  }
-
-  return async (root: ReaderRoot) => doRead(config, root);
-}
-
 export function readerFactory(component: unknown): IReader {
   validateReaderDefinition(component);
 

src/blocks/transformers/component/ComponentReaderOptions.tsx

@@ -19,7 +19,7 @@ type FrameworkOption = {
   detected?: FrameworkMeta;
 };
 
-export const readerOptions: FrameworkOption[] = [
+const readerOptions: FrameworkOption[] = [
   { value: "react", label: "React" },
   {
     // XXX: check if this needs to be vue or vuejs

src/blocks/transformers/component/TableReader.ts

@@ -26,10 +26,8 @@ import {
 import { findSingleElement } from "@/utils/requireSingleElement";
 import { lowerCase } from "lodash";
 
-export const TABLE_READER_ID = validateRegistryId("@pixiebrix/table-reader");
-export const TABLE_READER_ALL_ID = validateRegistryId(
-  "@pixiebrix/table-reader-all"
-);
+const TABLE_READER_ID = validateRegistryId("@pixiebrix/table-reader");
+const TABLE_READER_ALL_ID = validateRegistryId("@pixiebrix/table-reader-all");
 
 export class TableReader extends Transformer {
   constructor() {

src/blocks/types.ts

@@ -29,7 +29,7 @@ import { type UnknownObject } from "@/types";
  * @see https://developer.mozilla.org/en-US/docs/Web/API/URL_Pattern_API
  * @since 1.4.10
  */
-export type URLPattern = string | URLPatternInit;
+type URLPattern = string | URLPatternInit;
 
 /**
  * An availability rule. For a rule to match, there must be match from each of the provided entries.

src/chrome.ts

@@ -41,6 +41,7 @@ export type ReduxStorageKey = string & {
   _reduxStorageKeyBrand: never;
 };
 
+// Used only in the app
 export function setChromeExtensionId(extensionId = ""): void {
   forbidContext("extension");
 
@@ -52,6 +53,7 @@ export function setChromeExtensionId(extensionId = ""): void {
   }
 }
 
+// Used only in the app
 export function getChromeExtensionId(): string {
   forbidContext("extension");
 

src/components/BrickIcon.tsx

@@ -43,10 +43,7 @@ import { type MarketplaceListing } from "@/types/contract";
 import getType from "@/runtime/getType";
 import { type BlockType } from "@/runtime/runtimeTypes";
 
-export function getDefaultBrickIcon(
-  brick: IBrick,
-  blockType: BlockType
-): IconProp {
+function getDefaultBrickIcon(brick: IBrick, blockType: BlockType): IconProp {
   if ("schema" in brick) {
     return faCloud;
   }

src/components/ConfirmationModal.tsx

@@ -43,7 +43,7 @@ const initialModalState: ModalContextProps = {
   },
 };
 
-export const ModalContext = createContext<ModalContextProps>(initialModalState);
+const ModalContext = createContext<ModalContextProps>(initialModalState);
 
 const ConfirmationModal: React.FunctionComponent<
   ModalProps & {

src/components/addBlockModal/BlockGridItem.tsx

@@ -28,7 +28,7 @@ import Alert from "@/components/Alert";
 
 export const BLOCK_ITEM_FIXED_HEIGHT_PX = 89;
 
-export type BlockItemProps = {
+type BlockItemProps = {
   block: BlockResult;
   onSelect: () => void;
   onShowDetail: () => void;

src/components/documentBuilder/documentBuilderTypes.ts

@@ -52,7 +52,7 @@ export type DocumentElement<
   children?: DocumentElement[];
 };
 
-export type ListDocumentConfig = {
+type ListDocumentConfig = {
   array: Expression;
   elementKey?: string;
   element: DeferExpression<DocumentElement>;
@@ -69,7 +69,7 @@ export type PipelineDocumentConfig = {
   label: string;
   pipeline: PipelineExpression;
 };
-export type PipelineDocumentElement = DocumentElement<
+type PipelineDocumentElement = DocumentElement<
   "pipeline",
   PipelineDocumentConfig
 >;