Remove legacy "autoReadPattern" slack.

connectors/migrations/db/migration_43.sql

@@ -0,0 +1,2 @@
+-- Migration created on Jan 06, 2025
+ALTER TABLE slack_configurations DROP COLUMN "autoReadChannelPattern";

connectors/src/connectors/slack/index.ts

@@ -90,7 +90,6 @@ export class SlackConnectorManager extends BaseConnectorManager<SlackConfigurati
         dataSourceId: dataSourceConfig.dataSourceId,
       },
       {
-        autoReadChannelPattern: configuration.autoReadChannelPattern,
         autoReadChannelPatterns: configuration.autoReadChannelPatterns,
         botEnabled: configuration.botEnabled,
         slackTeamId: teamInfo.team.id,
@@ -709,10 +708,6 @@ export class SlackConnectorManager extends BaseConnectorManager<SlackConfigurati
         }
       }
 
-      case "autoReadChannelPattern": {
-        return new Err(new Error("autoReadChannelPattern is not deprecated"));
-      }
-
       case "autoReadChannelPatterns": {
         const parsedConfig = safeParseJSON(configValue);
         if (parsedConfig.isErr()) {
@@ -764,14 +759,6 @@ export class SlackConnectorManager extends BaseConnectorManager<SlackConfigurati
         return new Ok(botEnabledRes.value.toString());
       }
 
-      case "autoReadChannelPattern": {
-        const autoReadChannelPattern = await getAutoReadChannelPattern(
-          this.connectorId
-        );
-
-        return autoReadChannelPattern;
-      }
-
       case "autoReadChannelPatterns": {
         const autoReadChannelPatterns = await getAutoReadChannelPatterns(
           this.connectorId
@@ -878,24 +865,6 @@ export async function uninstallSlack(connectionId: string) {
   return new Ok(undefined);
 }
 
-export async function getAutoReadChannelPattern(
-  connectorId: ModelId
-): Promise<Result<string | null, Error>> {
-  const slackConfiguration =
-    await SlackConfigurationResource.fetchByConnectorId(connectorId);
-  if (!slackConfiguration) {
-    return new Err(
-      new Error(
-        `Failed to find a Slack configuration for connector ${connectorId}`
-      )
-    );
-  }
-  if (!slackConfiguration.autoReadChannelPattern) {
-    return new Ok(null);
-  }
-  return new Ok(slackConfiguration.autoReadChannelPattern);
-}
-
 export async function getAutoReadChannelPatterns(
   connectorId: ModelId
 ): Promise<Result<string | null, Error>> {

connectors/src/lib/models/slack.ts

@@ -18,7 +18,6 @@ export class SlackConfigurationModel extends BaseModel<SlackConfigurationModel>
   declare connectorId: ForeignKey<ConnectorModel["id"]>;
   // Whitelisted domains are in the format "domain:group_id".
   declare whitelistedDomains?: readonly string[];
-  declare autoReadChannelPattern?: string | null;
   declare autoReadChannelPatterns: SlackAutoReadPattern[];
 }
 
@@ -47,11 +46,6 @@ SlackConfigurationModel.init(
       type: DataTypes.ARRAY(DataTypes.STRING),
       allowNull: true,
     },
-    // TODO(2025-01-02 AutoReadCleanUp) Remove once fully migrated to `autoReadChannelPatterns`.
-    autoReadChannelPattern: {
-      type: DataTypes.STRING,
-      allowNull: true,
-    },
     autoReadChannelPatterns: {
       type: DataTypes.JSONB,
       allowNull: true,

types/src/connectors/slack.ts

@@ -21,8 +21,6 @@ export function isSlackAutoReadPatterns(
 export const SlackConfigurationTypeSchema = t.type({
   botEnabled: t.boolean,
   whitelistedDomains: t.union([t.array(t.string), t.undefined]),
-  // TODO(2025-01-02 AutoReadCleanUp) Remove once fully migrated to `autoReadChannelPatterns`.
-  autoReadChannelPattern: t.union([t.string, t.null, t.undefined]),
   autoReadChannelPatterns: SlackAutoReadPatternsSchema,
 });