Revert "fix: update aegir and revert varint"

This reverts commit d0a413f.

Author: hugomrdias

.github/workflows/main.yml

@@ -12,12 +12,13 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v2
-    - run: npm install
-    - run: npx aegir lint
-    - uses: gozala/[email protected].8
-    - run: npx aegir build
-    - run: npx aegir dep-check
+    - run: yarn
+    - run: yarn lint
+    - uses: gozala/[email protected].4
+    - run: yarn build
+    - run: yarn aegir dep-check
     - uses: ipfs/aegir/actions/bundle-size@master
+      name: size
       with:
         github_token: ${{ secrets.GITHUB_TOKEN }}
   test-node:
@@ -33,45 +34,34 @@ jobs:
       - uses: actions/setup-node@v1
         with:
           node-version: ${{ matrix.node }}
-      - run: npm install
-      - run: npx aegir test -t node --bail --cov
+      - run: yarn
+      - run: npx nyc --reporter=lcov aegir test -t node -- --bail
       - uses: codecov/codecov-action@v1
   test-chrome:
     needs: check
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - uses: microsoft/playwright-github-action@v1
-      - run: npm install
-      - run: npx aegir test -t browser -t webworker --bail --cov
-      - uses: codecov/codecov-action@v1
+      - run: yarn
+      - run: npx aegir test -t browser -t webworker --bail
   test-firefox:
     needs: check
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - uses: microsoft/playwright-github-action@v1
-      - run: npm install
-      - run: npx aegir test -t browser -t webworker --bail -- --browser firefox
-  test-webkit:
-    needs: check
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v2
-      - uses: microsoft/playwright-github-action@v1
-      - run: npm install
-      - run: npx aegir test -t browser -t webworker --bail -- --browser webkit
+      - run: yarn
+      - run: npx aegir test -t browser -t webworker --bail -- --browsers FirefoxHeadless
   test-electron-main:
     needs: check
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - run: npm install
+      - run: yarn
       - run: npx xvfb-maybe aegir test -t electron-main --bail
   test-electron-renderer:
     needs: check
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - run: npm install
+      - run: yarn
       - run: npx xvfb-maybe aegir test -t electron-renderer --bail

package.json

@@ -23,6 +23,8 @@
     "release": "aegir release",
     "release-minor": "aegir release --type minor",
     "release-major": "aegir release --type major",
+    "coverage": "aegir coverage",
+    "coverage-publish": "aegir coverage --provider coveralls",
     "update-table": "node tools/update-table.js"
   },
   "pre-push": [
@@ -48,23 +50,14 @@
   },
   "homepage": "https://github.com/multiformats/js-multicodec#readme",
   "dependencies": {
-    "uint8arrays": "^2.1.3",
-    "varint": "^5.0.2"
+    "uint8arrays": "1.1.0",
+    "varint": "^6.0.0"
   },
   "devDependencies": {
-    "@types/varint": "^5.0.0",
-    "aegir": "^31.0.1",
+    "aegir": "^29.2.2",
     "bent": "^7.3.12",
     "pre-push": "~0.1.1"
   },
-  "eslintConfig": {
-    "extends": "ipfs"
-  },
-  "aegir": {
-    "build": {
-      "bundlesizeMax": "6.7kB"
-    }
-  },
   "contributors": [
     "Henrique Dias <[email protected]>",
     "David Dias <[email protected]>",

src/generated-table.js

@@ -104,7 +104,6 @@ const baseTable = Object.freeze({
   'udt': 0x012d,
   'utp': 0x012e,
   'unix': 0x0190,
-  'thread': 0x0196,
   'p2p': 0x01a5,
   'ipfs': 0x01a5,
   'https': 0x01bb,

src/generated-types.d.ts

@@ -49,7 +49,7 @@ function addPrefix (multicodecStrOrCode, data) {
  * @returns {Uint8Array}
  */
 function rmPrefix (data) {
-  varint.decode(/** @type {Buffer} */(data))
+  varint.decode(data)
   return data.slice(varint.decode.bytes)
 }
 
@@ -60,7 +60,7 @@ function rmPrefix (data) {
  * @returns {CodecName}
  */
 function getNameFromData (prefixedData) {
-  const code = /** @type {CodecCode} */(varint.decode(/** @type {Buffer} */(prefixedData)))
+  const code = varint.decode(prefixedData)
   const name = codeToName[code]
   if (name === undefined) {
     throw new Error(`Code "${code}" not found`)
@@ -99,7 +99,7 @@ function getCodeFromName (name) {
  * @returns {CodecCode}
  */
 function getCodeFromData (prefixedData) {
-  return /** @type {CodecCode} */(varint.decode(/** @type {Buffer} */(prefixedData)))
+  return varint.decode(prefixedData)
 }
 
 /**

src/generated-types.ts

@@ -3,8 +3,6 @@
 /** @typedef {import('./generated-types').ConstantCodeMap} ConstantCodeMap */
 /** @typedef {import('./generated-types').NameUint8ArrayMap} NameUint8ArrayMap */
 /** @typedef {import('./generated-types').CodeNameMap} CodeNameMap */
-/** @typedef {import('./generated-types').CodecName} CodecName */
-/** @typedef {import('./generated-types').CodecConstant} CodecConstant */
 
 const { baseTable } = require('./generated-table')
 const varintEncode = require('./util').varintEncode
@@ -13,17 +11,15 @@ const nameToVarint = /** @type {NameUint8ArrayMap} */ ({})
 const constantToCode = /** @type {ConstantCodeMap} */({})
 const codeToName = /** @type {CodeNameMap} */({})
 
-// eslint-disable-next-line guard-for-in
 for (const name in baseTable) {
-  const codecName = /** @type {CodecName} */(name)
-  const code = baseTable[codecName]
-  nameToVarint[codecName] = varintEncode(code)
+  const code = baseTable[name]
+  nameToVarint[name] = varintEncode(code)
 
-  const constant = /** @type {CodecConstant} */(codecName.toUpperCase().replace(/-/g, '_'))
+  const constant = name.toUpperCase().replace(/-/g, '_')
   constantToCode[constant] = code
 
   if (!codeToName[code]) {
-    codeToName[code] = codecName
+    codeToName[code] = name
   }
 }
 

src/index.js

@@ -11,16 +11,10 @@ module.exports = {
   varintEncode
 }
 
-/**
- * @param {Uint8Array} buf
- */
 function uint8ArrayToNumber (buf) {
   return parseInt(uint8ArrayToString(buf, 'base16'), 16)
 }
 
-/**
- * @param {number} num
- */
 function numberToUint8Array (num) {
   let hexString = num.toString(16)
   if (hexString.length % 2 === 1) {
@@ -29,16 +23,10 @@ function numberToUint8Array (num) {
   return uint8ArrayFromString(hexString, 'base16')
 }
 
-/**
- * @param {Uint8Array} input
- */
 function varintUint8ArrayEncode (input) {
   return Uint8Array.from(varint.encode(uint8ArrayToNumber(input)))
 }
 
-/**
- * @param {number} num
- */
 function varintEncode (num) {
   return Uint8Array.from(varint.encode(num))
 }

src/maps.js

@@ -61,9 +61,8 @@ describe('multicodec', () => {
   })
 
   it('returns all codec numbers from names', () => {
-    // eslint-disable-next-line guard-for-in
     for (const name in nameToCode) {
-      expect(multicodec.getCodeFromName(/** @type {CodecName} */(name))).to.eql(nameToCode[/** @type {CodecName} */(name)])
+      expect(multicodec.getCodeFromName(/** @type {CodecName} */(name))).to.eql(nameToCode[name])
     }
   })
 

src/util.js

@@ -32,17 +32,17 @@ const run = async () => {
 /**
  * Constant names for all available codecs
  */
-export type CodecConstant = ${names.map(n => `${n.toUpperCase().replace(/-/g, '_')}`).join(' | ')}
+export type CodecConstant = ${names.map(n => `${n.toUpperCase().replace(/-/g, '_')}`).join(' | ')};
 
 /**
  * Names for all available codecs
  */
-export type CodecName = ${names.join(' | ')}
+export type CodecName = ${names.join(' | ')};
 
 /**
  * Number for all available codecs
  */
-export type CodecCode = ${codes.join(' | ')}
+export type CodecCode = ${codes.join(' | ')};
 
 export type ConstantCodeMap = Record<CodecConstant, CodecCode>
 export type NameUint8ArrayMap = Record<CodecName, Uint8Array>
@@ -64,7 +64,7 @@ ${processed}
 module.exports = { baseTable }
 `
 
-  fs.writeFileSync(path.join(__dirname, '../src/generated-types.d.ts'), typesTemplate)
+  fs.writeFileSync(path.join(__dirname, '../src/generated-types.ts'), typesTemplate)
   fs.writeFileSync(path.join(__dirname, '../src/generated-table.js'), tableTemplate)
 }