Hyrum's law breaks my heart (#943)

* Hyrum's law breaks my heart

* Changelog

Author: inlined

CHANGELOG.md

@@ -0,0 +1 @@
+- Fix an error that broke `firebase emulators:start` on older CLIs

spec/common/providers/https.spec.ts

@@ -2,8 +2,8 @@ import { expect } from 'chai';
 import * as express from 'express';
 import * as firebase from 'firebase-admin';
 
+import { apps as appsNamespace } from '../../../src/apps';
 import * as https from '../../../src/common/providers/https';
-import { apps as appsNamespace } from '../../../src/v1/apps';
 import * as mocks from '../../fixtures/credential/key.json';
 import {
   expectedResponseHeaders,

spec/v1/apps.spec.ts

@@ -21,7 +21,7 @@
 // SOFTWARE.
 
 import { expect } from 'chai';
-import { apps as appsNamespace } from '../../src/v1/apps';
+import { apps as appsNamespace } from '../../src/apps';
 
 import * as firebase from 'firebase-admin';
 import * as _ from 'lodash';

spec/v1/cloud-functions.spec.ts

@@ -29,7 +29,7 @@ import {
   EventContext,
   makeCloudFunction,
   MakeCloudFunctionArgs,
-} from '../../src/v1/cloud-functions';
+} from '../../src/cloud-functions';
 
 describe('makeCloudFunction', () => {
   const cloudFunctionArgs: MakeCloudFunctionArgs<any> = {

spec/v1/config.spec.ts

@@ -25,7 +25,7 @@ import * as fs from 'fs';
 import * as process from 'process';
 import Sinon = require('sinon');
 
-import * as config from '../../src/v1/config';
+import * as config from '../../src/config';
 
 describe('config()', () => {
   let readFileSync: Sinon.SinonStub;

spec/v1/function-builder.spec.ts

@@ -22,7 +22,7 @@
 
 import { expect } from 'chai';
 
-import * as functions from '../../src/v1';
+import * as functions from '../../src';
 
 describe('FunctionBuilder', () => {
   before(() => {

spec/v1/providers/analytics.spec.input.ts

@@ -21,7 +21,7 @@
 // SOFTWARE.
 
 /* tslint:disable:max-line-length */
-import { AnalyticsEvent } from '../../../src/v1/providers/analytics';
+import { AnalyticsEvent } from '../../../src/providers/analytics';
 
 // A payload, as it might arrive over the wire. Every possible field is filled out at least once.
 export const fullPayload = JSON.parse(`{

spec/v1/providers/analytics.spec.ts

@@ -22,9 +22,9 @@
 
 import { expect } from 'chai';
 
-import * as functions from '../../../src/v1';
-import { Event, EventContext } from '../../../src/v1/cloud-functions';
-import * as analytics from '../../../src/v1/providers/analytics';
+import * as functions from '../../../src';
+import { Event, EventContext } from '../../../src/cloud-functions';
+import * as analytics from '../../../src/providers/analytics';
 import * as analytics_spec_input from './analytics.spec.input';
 
 describe('Analytics Functions', () => {

spec/v1/providers/auth.spec.ts

@@ -23,13 +23,13 @@
 import { expect } from 'chai';
 import * as firebase from 'firebase-admin';
 
-import * as functions from '../../../src/index';
 import {
   CloudFunction,
   Event,
   EventContext,
-} from '../../../src/v1/cloud-functions';
-import * as auth from '../../../src/v1/providers/auth';
+} from '../../../src/cloud-functions';
+import * as functions from '../../../src/index';
+import * as auth from '../../../src/providers/auth';
 
 describe('Auth Functions', () => {
   const event: Event = {

spec/v1/providers/database.spec.ts

@@ -21,11 +21,11 @@
 // SOFTWARE.
 
 import { expect } from 'chai';
-import { apps as appsNamespace } from '../../../src/v1/apps';
-import * as config from '../../../src/v1/config';
-import * as functions from '../../../src/v1/index';
-import * as database from '../../../src/v1/providers/database';
-import { applyChange } from '../../../src/v1/utils';
+import { apps as appsNamespace } from '../../../src/apps';
+import * as config from '../../../src/config';
+import * as functions from '../../../src/index';
+import * as database from '../../../src/providers/database';
+import { applyChange } from '../../../src/utils';
 
 describe('Database Functions', () => {
   describe('DatabaseBuilder', () => {

spec/v1/providers/firestore.spec.ts

@@ -25,7 +25,7 @@ import * as admin from 'firebase-admin';
 import * as _ from 'lodash';
 
 import * as functions from '../../../src/index';
-import * as firestore from '../../../src/v1/providers/firestore';
+import * as firestore from '../../../src/providers/firestore';
 
 describe('Firestore Functions', () => {
   function constructValue(fields: any) {

spec/v1/providers/https.spec.ts

@@ -24,7 +24,7 @@ import { expect } from 'chai';
 import * as express from 'express';
 import * as _ from 'lodash';
 import * as functions from '../../../src/index';
-import * as https from '../../../src/v1/providers/https';
+import * as https from '../../../src/providers/https';
 import {
   expectedResponseHeaders,
   MockRequest,

spec/v1/providers/pubsub.spec.ts

@@ -23,7 +23,7 @@
 import { expect } from 'chai';
 import { Event } from '../../../src/index';
 import * as functions from '../../../src/index';
-import * as pubsub from '../../../src/v1/providers/pubsub';
+import * as pubsub from '../../../src/providers/pubsub';
 
 describe('Pubsub Functions', () => {
   describe('pubsub.Message', () => {

spec/v1/providers/remoteConfig.spec.ts

@@ -22,14 +22,14 @@
 import { expect } from 'chai';
 import * as _ from 'lodash';
 
-import * as functions from '../../../src/index';
 import {
   CloudFunction,
   Event,
   EventContext,
   TriggerAnnotated,
-} from '../../../src/v1/cloud-functions';
-import * as remoteConfig from '../../../src/v1/providers/remoteConfig';
+} from '../../../src/cloud-functions';
+import * as functions from '../../../src/index';
+import * as remoteConfig from '../../../src/providers/remoteConfig';
 
 describe('RemoteConfig Functions', () => {
   function constructVersion() {

spec/v1/providers/storage.spec.ts

@@ -21,10 +21,10 @@
 // SOFTWARE.
 
 import { expect } from 'chai';
-import { Event, EventContext } from '../../../src/v1';
-import * as functions from '../../../src/v1';
-import * as config from '../../../src/v1/config';
-import * as storage from '../../../src/v1/providers/storage';
+import { Event, EventContext } from '../../../src';
+import * as functions from '../../../src';
+import * as config from '../../../src/config';
+import * as storage from '../../../src/providers/storage';
 
 describe('Storage Functions', () => {
   describe('ObjectBuilder', () => {

spec/v1/providers/testLab.spec.ts

@@ -22,7 +22,7 @@
 
 import { expect } from 'chai';
 
-import * as testLab from '../../../src/v1/providers/testLab';
+import * as testLab from '../../../src/providers/testLab';
 
 describe('Test Lab Functions', () => {
   describe('#onComplete', () => {

spec/v1/setup.spec.ts

@@ -21,7 +21,7 @@
 // SOFTWARE.
 
 import { expect } from 'chai';
-import { setup } from '../../src/v1/setup';
+import { setup } from '../../src/setup';
 
 describe('setup()', () => {
   afterEach(() => {

spec/v1/utils.spec.ts

@@ -21,7 +21,7 @@
 // SOFTWARE.
 
 import { expect } from 'chai';
-import { applyChange } from '../../src/v1/utils';
+import { applyChange } from '../../src/utils';
 
 describe('utils', () => {
   describe('.applyChange(from: any, to: any): any', () => {

src/v1/apps.ts renamed to src/apps.ts

@@ -22,22 +22,22 @@
 
 import { Request, Response } from 'express';
 import * as _ from 'lodash';
-import { warn } from '../logger';
 import {
   DEFAULT_FAILURE_POLICY,
   DeploymentOptions,
   FailurePolicy,
   Schedule,
 } from './function-configuration';
+import { warn } from './logger';
 export { Request, Response };
 import {
   convertIfPresent,
+  convertInvoker,
   copyIfPresent,
   Duration,
   durationFromSeconds,
   serviceAccountFromShorthand,
-  convertInvoker,
-} from '../common/encoding';
+} from './common/encoding';
 
 /** @hidden */
 const WILDCARD_REGEX = new RegExp('{[^/{}]*}', 'g');

src/v1/cloud-functions.ts renamed to src/cloud-functions.ts

@@ -28,7 +28,7 @@ import * as logger from '../../logger';
 
 // TODO(inlined): Decide whether we want to un-version apps or whether we want a
 // different strategy
-import { apps } from '../../v1/apps';
+import { apps } from '../../apps';
 
 /** @hidden */
 export interface Request extends express.Request {

src/common/providers/https.ts

@@ -1,6 +1,6 @@
 // The MIT License (MIT)
 //
-// Copyright (c) 2021 Firebase
+// Copyright (c) 2017 Firebase
 //
 // Permission is hereby granted, free of charge, to any person obtaining a copy
 // of this software and associated documentation files (the "Software"), to deal
@@ -20,4 +20,43 @@
 // OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 // SOFTWARE.
 
-export * from './v1';
+// Providers:
+import * as analytics from './providers/analytics';
+import * as auth from './providers/auth';
+import * as database from './providers/database';
+import * as firestore from './providers/firestore';
+import * as https from './providers/https';
+import * as pubsub from './providers/pubsub';
+import * as remoteConfig from './providers/remoteConfig';
+import * as storage from './providers/storage';
+import * as testLab from './providers/testLab';
+
+import * as apps from './apps';
+import { handler } from './handler-builder';
+import * as logger from './logger';
+import { setup } from './setup';
+
+const app = apps.apps();
+
+export {
+  analytics,
+  app,
+  auth,
+  database,
+  firestore,
+  handler,
+  https,
+  pubsub,
+  remoteConfig,
+  storage,
+  testLab,
+  logger,
+};
+
+// Exported root types:
+export * from './cloud-functions';
+export * from './config';
+export * from './function-builder';
+export * from './function-configuration';
+
+setup();

src/v1/config.ts renamed to src/config.ts

@@ -22,7 +22,6 @@
 
 import * as firebase from 'firebase-admin';
 import * as _ from 'lodash';
-import { joinPath, normalizePath, pathParts } from '../../utilities/path';
 import { apps } from '../apps';
 import {
   Change,
@@ -33,6 +32,7 @@ import {
 } from '../cloud-functions';
 import { firebaseConfig } from '../config';
 import { DeploymentOptions } from '../function-configuration';
+import { joinPath, normalizePath, pathParts } from '../utilities/path';
 import { applyChange } from '../utils';
 
 /** @hidden */

src/v1/encoder.ts renamed to src/encoder.ts

@@ -24,7 +24,6 @@ import * as firebase from 'firebase-admin';
 import * as _ from 'lodash';
 
 import { posix } from 'path';
-import * as logger from '../../logger';
 import { apps } from '../apps';
 import {
   Change,
@@ -35,6 +34,7 @@ import {
 } from '../cloud-functions';
 import { dateToTimestampProto } from '../encoder';
 import { DeploymentOptions } from '../function-configuration';
+import * as logger from '../logger';
 
 /** @hidden */
 export const provider = 'google.firestore';

src/v1/function-builder.ts renamed to src/function-builder.ts

@@ -22,8 +22,8 @@
 
 import * as express from 'express';
 
-import * as common from '../../common/providers/https';
 import { HttpsFunction, optionsToTrigger, Runnable } from '../cloud-functions';
+import * as common from '../common/providers/https';
 import { DeploymentOptions } from '../function-configuration';
 
 export type Request = common.Request;

src/v1/function-configuration.ts renamed to src/function-configuration.ts

@@ -21,8 +21,8 @@
 // SOFTWARE.
 
 /** @hidden */
-import { warn } from '../logger';
 import { firebaseConfig } from './config';
+import { warn } from './logger';
 
 // Set up for config and vars
 export function setup() {