remove exp references

Author: Feiyang1

.github/CODEOWNERS

@@ -35,6 +35,7 @@
 
 # Database Code
 packages/database @schmidt-sebastian @jsdt @firebase/jssdk-global-approvers
+packages/database-compat @schmidt-sebastian @jsdt @firebase/jssdk-global-approvers
 packages/database-types @schmidt-sebastian @jsdt @firebase/jssdk-global-approvers
 
 # Firestore Code
@@ -45,56 +46,54 @@ integration/firestore  @firebase/firestore-js-team @firebase/jssdk-global-approv
 
 # Storage Code
 packages/storage  @schmidt-sebastian @firebase/jssdk-global-approvers
+packages/storage-compat  @schmidt-sebastian @firebase/jssdk-global-approvers
 packages/storage-types  @schmidt-sebastian @firebase/jssdk-global-approvers
 
 # Messaging Code
 packages/messaging  @zwu52 @chliangGoogle @ciarand @firebase/jssdk-global-approvers
+packages/messaging-compat  @zwu52 @chliangGoogle @ciarand @firebase/jssdk-global-approvers
 packages/messaging-types @zwu52 @chliangGoogle @ciarand @firebase/jssdk-global-approvers
+packages/messaging-interop-types @zwu52 @chliangGoogle @ciarand @firebase/jssdk-global-approvers
 integration/messaging @zwu52 @chliangGoogle @ciarand @firebase/jssdk-global-approvers
 
 # Auth Code
 packages/auth  @bojeil-google @avolkovi @sam-gc @yuchenshi @firebase/jssdk-global-approvers
+packages/auth-compat @avolkovi @sam-gc @yuchenshi @firebase/jssdk-global-approvers
 packages/auth-types  @bojeil-google @avolkovi @sam-gc @yuchenshi @firebase/jssdk-global-approvers
+packages/auth-interop-types  @bojeil-google @avolkovi @sam-gc @yuchenshi @firebase/jssdk-global-approvers
 
 # Testing Code
-packages/testing  @avolkovi @sam-gc @yuchenshi @firebase/jssdk-global-approvers
 packages/rules-unit-testing  @avolkovi @sam-gc @yuchenshi @firebase/jssdk-global-approvers
 
 # Installations
-packages/installations @andirayo @ChaoqunCHEN @firebase/jssdk-global-approvers
-packages/installations-types @andirayo @ChaoqunCHEN @firebase/jssdk-global-approvers
+packages/installations @avolkovi @yoyomyo @firebase/jssdk-global-approvers
+packages/installations-compat @avolkovi @yoyomyo @firebase/jssdk-global-approvers
+packages/installations-types @avolkovi @yoyomyo @firebase/jssdk-global-approvers
 
 # Performance Code
-packages/performance  @alikn @zijianjoy @firebase/jssdk-global-approvers
-packages/performance-types  @alikn @zijianjoy @firebase/jssdk-global-approvers
+packages/performance @jposuna @firebase/jssdk-global-approvers
+packages/performance-compat @jposuna @firebase/jssdk-global-approvers
+packages/performance-types @jposuna @firebase/jssdk-global-approvers
 
 # Analytics Code
 packages/analytics @hsubox76 @Feiyang1 @firebase/jssdk-global-approvers
+packages/analytics-compat @hsubox76 @Feiyang1 @firebase/jssdk-global-approvers
 packages/analytics-types @hsubox76 @Feiyang1 @firebase/jssdk-global-approvers
 
 # Remote Config Code
 packages/remote-config @erikeldridge @firebase/jssdk-global-approvers
+packages/remote-config-compat  @erikeldridge @firebase/jssdk-global-approvers
 packages/remote-config-types @erikeldridge @firebase/jssdk-global-approvers
 
+# App Check Code
+packages/app-check @hsubox76 @Feiyang1 @firebase/jssdk-global-approvers
+packages/app-check-compat @hsubox76 @Feiyang1 @firebase/jssdk-global-approvers
+packages/app-check-types @hsubox76 @Feiyang1 @firebase/jssdk-global-approvers
+packages/app-check-interop-types @hsubox76 @Feiyang1 @firebase/jssdk-global-approvers
+
 # Documentation Changes
 packages/firebase/index.d.ts @egilmorez @firebase/jssdk-global-approvers
 scripts/docgen/content-sources/ @egilmorez @firebase/jssdk-global-approvers
 
 # Changeset
 .changeset @egilmorez @firebase/jssdk-changeset-approvers @firebase/firestore-js-team @firebase/jssdk-global-approvers
-
-# Auth-Exp Code
-packages-exp/auth-exp  @avolkovi @sam-gc @yuchenshi @firebase/jssdk-global-approvers
-packages-exp/auth-compat-exp @avolkovi @sam-gc @yuchenshi @firebase/jssdk-global-approvers
-
-# Installations-Exp Code
-packages/installations-exp @avolkovi @yoyomyo @firebase/jssdk-global-approvers
-packages/installations-types-exp @avolkovi @yoyomyo @firebase/jssdk-global-approvers
-
-# Perf-Exp Code
-packages/performance-exp  @alikn @zijianjoy @firebase/jssdk-global-approvers
-packages/performance-types-exp  @alikn @zijianjoy @firebase/jssdk-global-approvers
-
-# RC-Exp Code
-packages/remote-config-exp  @erikeldridge @firebase/jssdk-global-approvers
-packages/remote-config-compat  @erikeldridge @firebase/jssdk-global-approvers

.gitignore

@@ -84,10 +84,8 @@ package-lock.json
 
 # temp folder used by api-extractor
 temp
-packages-exp/**/temp
 
 # temp markdowns generated for individual SDKs
-packages-exp/**/docs
 packages/**/docs
 
 # files generated by api-extractor that should not be tracked

lerna.json

@@ -3,7 +3,6 @@
   "npmClient": "yarn",
   "packages": [
     "packages/*",
-    "packages-exp/*",
     "integration/*",
     "repo-scripts/*"
   ],

package.json

@@ -46,7 +46,7 @@
     "lint:fix": "lerna run --scope @firebase/* lint:fix",
     "size-report": "ts-node-script scripts/size_report/report_binary_size.ts",
     "modular-export-size-report": "ts-node-script scripts/size_report/report_modular_export_binary_size.ts",
-    "api-report": "lerna run --scope @firebase/*-exp --scope @firebase/firestore --scope @firebase/storage --scope @firebase/storage-types --scope @firebase/database api-report",
+    "api-report": "lerna run --scope @firebase/* api-report",
     "docgen:exp": "ts-node-script scripts/exp/docgen.ts",
     "postinstall": "yarn --cwd repo-scripts/changelog-generator build",
     "sa": "ts-node-script repo-scripts/size-analysis/cli.ts",
@@ -58,7 +58,6 @@
   },
   "workspaces": [
     "packages/*",
-    "packages-exp/*",
     "integration/*",
     "repo-scripts/*"
   ],

packages/analytics/rollup.config.release.js

@@ -22,7 +22,6 @@
     "test:browser": "karma start --single-run",
     "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha src/**/*.test.ts --config ../../config/mocharc.node.js",
     "api-report": "api-extractor run --local --verbose",
-    "predoc": "node ../../scripts/exp/remove-exp.js temp",
     "doc": "api-documenter markdown --input temp --output docs",
     "build:doc": "yarn build && yarn doc",
     "typings:public": "node ../../scripts/exp/use_typings.js ./dist/app-public.d.ts",

packages/app/package.json

@@ -69,9 +69,8 @@ export async function analyzePackageSize(
     writeReportToFile(jsonReport, resolve(argv.output));
   } else if (!argv.inputDtsFile && !argv.inputBundleFile) {
     // retrieve All Module Names
-    // TODO: update the workspace once exp packages are officially released
     let allModulesLocation = await mapWorkspaceToPackages([
-      `${projectRoot}/packages-exp/*`
+      `${projectRoot}/packages/*`
     ]);
     allModulesLocation = allModulesLocation.filter(path => {
       const json = JSON.parse(

repo-scripts/size-analysis/package-analysis.ts

@@ -79,7 +79,7 @@ async function getDiffData(): Promise<{
       changesetFile = changesetMatch[0];
     }
     // Check for changed files inside package dirs.
-    const pkgMatch = filename.match('^(packages(-exp)?/[a-zA-Z0-9-]+)/.*');
+    const pkgMatch = filename.match('^(packages/[a-zA-Z0-9-]+)/.*');
     if (pkgMatch && pkgMatch[1]) {
       // skip packages without package.json
       // It could happen when we rename a package or remove a package from the repo

scripts/check_changeset.ts

@@ -18,7 +18,6 @@
 import { spawn } from 'child-process-promise';
 import { mapWorkspaceToPackages } from '../release/utils/workspace';
 import { projectRoot } from '../utils';
-import { removeExpSuffix } from './remove-exp';
 import fs from 'fs';
 import glob from 'glob';
 import * as yargs from 'yargs';
@@ -39,38 +38,10 @@ async function generateDocs(forDevsite: boolean = false) {
   const outputFolder = forDevsite ? 'docs-devsite' : 'docs-exp';
   const command = forDevsite ? 'api-documenter-devsite' : 'api-documenter';
 
-  // TODO: change yarn command once exp packages become official
-  await spawn('yarn', ['lerna', 'run', '--scope', '@firebase/*-exp', 'build'], {
+  await spawn('yarn', ['lerna', 'run', '--scope', '@firebase/*', 'build'], {
     stdio: 'inherit'
   });
 
-  // build storage-exp
-  await spawn(
-    'yarn',
-    ['lerna', 'run', '--scope', '@firebase/storage', 'build:exp'],
-    {
-      stdio: 'inherit'
-    }
-  );
-
-  // build database-exp
-  await spawn(
-    'yarn',
-    ['lerna', 'run', '--scope', '@firebase/database', 'build:exp'],
-    {
-      stdio: 'inherit'
-    }
-  );
-
-  // generate public typings for firestore
-  await spawn(
-    'yarn',
-    ['lerna', 'run', '--scope', '@firebase/firestore', 'prebuild'],
-    {
-      stdio: 'inherit'
-    }
-  );
-
   await spawn('yarn', ['api-report'], {
     stdio: 'inherit'
   });
@@ -82,8 +53,7 @@ async function generateDocs(forDevsite: boolean = false) {
   // TODO: Throw error if path doesn't exist once all packages add markdown support.
   const apiJsonDirectories = (
     await mapWorkspaceToPackages([
-      `${projectRoot}/packages/*`,
-      `${projectRoot}/packages-exp/*`
+      `${projectRoot}/packages/*`
     ])
   )
     .map(path => `${path}/temp`)
@@ -106,8 +76,6 @@ async function generateDocs(forDevsite: boolean = false) {
     fs.copyFileSync(paths[0], `${tmpDir}/${fileName}`);
   }
 
-  // Generate docs without the -exp suffix
-  removeExpSuffix(tmpDir);
   await spawn(
     'yarn',
     [command, 'markdown', '--input', 'temp', '--output', outputFolder],

scripts/exp/docgen.ts

@@ -41,7 +41,7 @@ export async function getChangedPackages(
   const files = changedFiles || (await getChangedFiles());
   for (const filename of files) {
     // Check for changed files inside package dirs.
-    const match = filename.match('^(packages(-exp)?/[a-zA-Z0-9-]+)/.*');
+    const match = filename.match('^(packages/[a-zA-Z0-9-]+)/.*');
     if (match && match[1]) {
       const changedPackage = require(resolve(
         projectRoot,