Ratchet up linting for functions package a little bit (#431)

functions/.eslintrc.js

@@ -9,17 +9,27 @@ module.exports = {
     {
       files: ['**/*.ts'],
       plugins: ['@typescript-eslint'],
+      extends: ['plugin:@typescript-eslint/recommended'],
       parser: '@typescript-eslint/parser',
       parserOptions: {
         tsconfigRootDir: __dirname,
         project: ['./tsconfig.json'],
       },
+      rules: {
+        '@typescript-eslint/no-explicit-any': 'off',
+        '@typescript-eslint/no-extra-semi': 'off',
+        '@typescript-eslint/no-unused-vars': [
+          'error',
+          {
+            argsIgnorePattern: '^_',
+            varsIgnorePattern: '^_',
+            caughtErrorsIgnorePattern: '^_',
+          },
+        ],
+      },
     },
   ],
   rules: {
-    'no-extra-semi': 'off',
-    'no-unused-vars': 'off',
-    'no-constant-condition': ['error', { checkLoops: false }],
     'lodash/import-scope': [2, 'member'],
   },
 }

functions/src/api.ts

@@ -27,7 +27,6 @@ export class APIError {
     this.msg = msg
     this.details = details
   }
-  toJson() {}
 }
 
 export const parseCredentials = async (req: Request): Promise<Credentials> => {

functions/src/backup-db.ts

@@ -24,9 +24,12 @@ const bucket = 'gs://manifold-firestore-backup'
 
 export const backupDb = functions.pubsub
   .schedule('every 24 hours')
-  .onRun((context) => {
+  .onRun((_context) => {
     const projectId = process.env.GCP_PROJECT || process.env.GCLOUD_PROJECT
-    const databaseName = client.databasePath(projectId!, '(default)')
+    if (projectId == null) {
+      throw new Error('No project ID environment variable set.')
+    }
+    const databaseName = client.databasePath(projectId, '(default)')
 
     return client
       .exportDocuments({

functions/src/call-cloud-function.ts

@@ -2,10 +2,10 @@ import * as admin from 'firebase-admin'
 
 import fetch from './fetch'
 
-export const callCloudFunction = (functionName: string, data: {} = {}) => {
+export const callCloudFunction = (functionName: string, data: unknown = {}) => {
   const projectId = admin.instanceId().app.options.projectId
 
-  let url = `https://us-central1-${projectId}.cloudfunctions.net/${functionName}`
+  const url = `https://us-central1-${projectId}.cloudfunctions.net/${functionName}`
 
   return fetch(url, {
     method: 'POST',

functions/src/create-fold.ts

@@ -22,17 +22,18 @@ export const createFold = functions.runWith({ minInstances: 1 }).https.onCall(
     const creator = await getUser(userId)
     if (!creator) return { status: 'error', message: 'User not found' }
 
-    let { name, about, tags } = data
+    let { name, about } = data
 
     if (!name || typeof name !== 'string')
       return { status: 'error', message: 'Name must be a non-empty string' }
-
     name = name.trim().slice(0, 140)
 
     if (typeof about !== 'string')
       return { status: 'error', message: 'About must be a string' }
     about = about.trim().slice(0, 140)
 
+    const { tags } = data
+
     if (!Array.isArray(tags))
       return { status: 'error', message: 'Tags must be an array of strings' }
 

functions/src/create-notification.ts

@@ -72,7 +72,7 @@ export const createNotification = async (
     sourceType === 'answer' ||
     sourceType === 'contract'
   ) {
-    let reasonTextPretext = getReasonTextFromReason(sourceType, reason)
+    const reasonTextPretext = getReasonTextFromReason(sourceType, reason)
 
     const notifyContractCreator = async (
       userToReasonTexts: user_to_reason_texts

functions/src/emails.ts

@@ -3,7 +3,7 @@ import { Answer } from '../../common/answer'
 import { Bet } from '../../common/bet'
 import { getProbability } from '../../common/calculate'
 import { Comment } from '../../common/comment'
-import { Contract, FreeResponseContract } from '../../common/contract'
+import { Contract } from '../../common/contract'
 import { DPM_CREATOR_FEE } from '../../common/fees'
 import { PrivateUser, User } from '../../common/user'
 import { formatMoney, formatPercent } from '../../common/util/format'

functions/src/market-close-emails.ts

@@ -40,7 +40,7 @@ async function sendMarketCloseEmails() {
       .filter((x) => !!x) as Contract[]
   })
 
-  for (let contract of contracts) {
+  for (const contract of contracts) {
     console.log(
       'sending close email for',
       contract.slug,

functions/src/on-fold-delete.ts

@@ -2,7 +2,7 @@ import * as functions from 'firebase-functions'
 
 export const onFoldDelete = functions.firestore
   .document('folds/{foldId}')
-  .onDelete(async (change, context) => {
+  .onDelete(async (change, _context) => {
     const snapshot = await change.ref.collection('followers').get()
 
     // Delete followers sub-collection.

functions/src/scripts/cache-views.ts

@@ -52,25 +52,25 @@ async function cacheUserViews(userId: string) {
   console.log(viewCounts, lastViewTime)
 }
 
-async function deleteCache() {
+// async function deleteCache() {
-  console.log('Deleting view cache')
+//   console.log('Deleting view cache')
 
-  const users = await getValues<User>(firestore.collection('users'))
+//   const users = await getValues<User>(firestore.collection('users'))
 
-  await batchedWaitAll(
+//   await batchedWaitAll(
-    users.map((user) => async () => {
+//     users.map((user) => async () => {
-      console.log('Deleting view cache for', user.username)
+//       console.log('Deleting view cache for', user.username)
-      await firestore.doc(`private-users/${user.id}/cache/viewCounts`).delete()
+//       await firestore.doc(`private-users/${user.id}/cache/viewCounts`).delete()
-      await firestore
+//       await firestore
-        .doc(`private-users/${user.id}/cache/lastViewTime`)
+//         .doc(`private-users/${user.id}/cache/lastViewTime`)
-        .delete()
+//         .delete()
-      await firestore
+//       await firestore
-        .doc(`private-users/${user.id}/cache/contractScores`)
+//         .doc(`private-users/${user.id}/cache/contractScores`)
-        .delete()
+//         .delete()
-      await firestore.doc(`private-users/${user.id}/cache/wordScores`).delete()
+//       await firestore.doc(`private-users/${user.id}/cache/wordScores`).delete()
-    })
+//     })
-  )
+//   )
-}
+// }
 
 if (require.main === module) {
   cacheViews().then(() => process.exit())

functions/src/scripts/change-user-info.ts

@@ -1,5 +1,3 @@
-import * as admin from 'firebase-admin'
-
 import { initAdmin } from './script-init'
 initAdmin()
 

functions/src/scripts/create-private-users.ts

@@ -11,7 +11,7 @@ async function main() {
   const snap = await firestore.collection('users').get()
   const users = snap.docs.map((d) => d.data() as User)
 
-  for (let user of users) {
+  for (const user of users) {
     const fbUser = await admin.auth().getUser(user.id)
     const email = fbUser.email
     const { username } = user

functions/src/scripts/denormalize.ts

@@ -6,7 +6,7 @@ import { DocumentSnapshot, Transaction } from 'firebase-admin/firestore'
 export type DocumentValue = {
   doc: DocumentSnapshot
   field: string
-  val: any
+  val: unknown
 }
 export type DocumentCorrespondence = [DocumentSnapshot, DocumentSnapshot[]]
 export type DocumentDiff = {
@@ -20,9 +20,9 @@ export function findDiffs(
   destPath: string
 ) {
   const diffs: DocumentDiff[] = []
-  for (let [srcDoc, destDocs] of docs) {
+  for (const [srcDoc, destDocs] of docs) {
     const srcVal = srcDoc.get(srcPath)
-    for (let destDoc of destDocs) {
+    for (const destDoc of destDocs) {
       const destVal = destDoc.get(destPath)
       if (destVal !== srcVal) {
         diffs.push({

functions/src/scripts/migrate-contract.ts

@@ -18,7 +18,7 @@ async function migrateBet(contractRef: DocRef, bet: Bet) {
   await contractRef.collection('bets').doc(id).update({ shares })
 }
 
-async function migrateContract(contractRef: DocRef, contract: Contract) {
+async function migrateContract(contractRef: DocRef) {
   const bets = await contractRef
     .collection('bets')
     .get()
@@ -48,7 +48,7 @@ async function migrateContracts() {
     console.log('contract', contract.question, 'bets', bets.length)
 
     for (const bet of bets) await migrateBet(contractRef, bet)
-    await migrateContract(contractRef, contract)
+    await migrateContract(contractRef)
   }
 }
 

functions/src/scripts/migrate-to-cfmm.ts

@@ -57,7 +57,7 @@ async function recalculateContract(contractRef: DocRef, isCommit = false) {
         : 0
     }
 
-    for (let bet of bets) {
+    for (const bet of bets) {
       const shares = bet.sale
         ? getSoldBetPayout(bet)
         : bet.isSold
@@ -139,7 +139,7 @@ async function main() {
             !snap.empty ? [firestore.doc(`contracts/${snap.docs[0].id}`)] : []
           )
 
-  for (let contractRef of contractRefs) {
+  for (const contractRef of contractRefs) {
     await recalculateContract(contractRef, isCommit).catch((e) =>
       console.log('error: ', e, 'id=', contractRef.id)
     )

functions/src/scripts/migrate-to-dpm-2.ts

@@ -100,7 +100,7 @@ async function recalculateContract(
 
     console.log('start', { pool, totalBets, totalShares })
 
-    for (let bet of bets) {
+    for (const bet of bets) {
       if (bet.sale) {
         const soldBet = bets.find((b) => b.id === bet.sale?.betId)
         if (!soldBet) throw new Error('invalid sold bet' + bet.sale.betId)

functions/src/scripts/script-init.ts

@@ -64,6 +64,7 @@ export const initAdmin = (env?: string) => {
     return
   }
   console.log(`Initializing connection to ${env} Firebase...`)
+  /* eslint-disable-next-line @typescript-eslint/no-var-requires */
   const serviceAccount = require(keyPath)
   admin.initializeApp({
     credential: admin.credential.cert(serviceAccount),

functions/src/sell-bet.ts

@@ -6,7 +6,6 @@ import { User } from '../../common/user'
 import { Bet } from '../../common/bet'
 import { getSellBetInfo } from '../../common/sell-bet'
 import { addObjects, removeUndefinedProps } from '../../common/util/object'
-import { Fees } from '../../common/fees'
 
 export const sellBet = functions.runWith({ minInstances: 1 }).https.onCall(
   async (
@@ -50,7 +49,8 @@ export const sellBet = functions.runWith({ minInstances: 1 }).https.onCall(
       if (!betSnap.exists) return { status: 'error', message: 'Invalid bet' }
       const bet = betSnap.data() as Bet
 
-      if (userId !== bet.userId) return { status: 'error', message: 'Not authorized' }
+      if (userId !== bet.userId)
+        return { status: 'error', message: 'Not authorized' }
       if (bet.isSold) return { status: 'error', message: 'Bet already sold' }
 
       const newBetDoc = firestore

functions/src/unsubscribe.ts

@@ -6,7 +6,8 @@ import { PrivateUser } from '../../common/user'
 export const unsubscribe = functions
   .runWith({ minInstances: 1 })
   .https.onRequest(async (req, res) => {
-    let { id, type } = req.query as { id: string; type: string }
+    const id = req.query.id as string
+    let type = req.query.type as string
     if (!id || !type) {
       res.status(400).send('Empty id or type parameter.')
       return

functions/src/update-feed.ts

@@ -31,7 +31,7 @@ const MAX_BATCHES = 50
 
 const getUserBatches = async () => {
   const users = shuffle(await getValues<User>(firestore.collection('users')))
-  let userBatches: User[][] = []
+  const userBatches: User[][] = []
   for (let i = 0; i < users.length; i += BATCH_SIZE) {
     userBatches.push(users.slice(i, i + BATCH_SIZE))
   }
@@ -197,18 +197,18 @@ function getActivityScore(contract: Contract, viewTime: number | undefined) {
   return isNew ? newMappedScore : mappedScore
 }
 
-function getLastViewedScore(viewTime: number | undefined) {
+// function getLastViewedScore(viewTime: number | undefined) {
-  if (viewTime === undefined) {
+//   if (viewTime === undefined) {
-    return 1
+//     return 1
-  }
+//   }
 
-  const daysAgo = (Date.now() - viewTime) / DAY_MS
+//   const daysAgo = (Date.now() - viewTime) / DAY_MS
 
-  if (daysAgo < 0.5) {
+//   if (daysAgo < 0.5) {
-    const frac = logInterpolation(0, 0.5, daysAgo)
+//     const frac = logInterpolation(0, 0.5, daysAgo)
-    return 0.5 + 0.25 * frac
+//     return 0.5 + 0.25 * frac
-  }
+//   }
 
-  const frac = logInterpolation(0.5, 14, daysAgo)
+//   const frac = logInterpolation(0.5, 14, daysAgo)
-  return 0.75 + 0.25 * frac
+//   return 0.75 + 0.25 * frac
-}
+// }

functions/src/update-user-metrics.ts

@@ -71,9 +71,9 @@ const computeTotalPool = async (
   return sum(pools)
 }
 
-const computeVolume = async (contract: Contract) => {
+// const computeVolume = async (contract: Contract) => {
-  const bets = await getValues<Bet>(
+//   const bets = await getValues<Bet>(
-    firestore.collection(`contracts/${contract.id}/bets`)
+//     firestore.collection(`contracts/${contract.id}/bets`)
-  )
+//   )
-  return sumBy(bets, (bet) => Math.abs(bet.amount))
+//   return sumBy(bets, (bet) => Math.abs(bet.amount))
-}
+// }