Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix integration bugs with new schema #90

Merged
merged 7 commits into from
Aug 4, 2021
Merged

Fix integration bugs with new schema #90

merged 7 commits into from
Aug 4, 2021

Conversation

jacksteamdev
Copy link
Contributor

  • Upgraded AJV, TypeScript, Jest, etc
  • Fixes bugs preventing AJV from using new schema
  • Fix TS errors related to new TS version
  • Disable TS-Eslint rules, the upgraded packages were too big for their britches
  • Increase Jest timeout for pre-test builds during e2e tests

@github-actions
Copy link
Contributor

github-actions bot commented Aug 4, 2021

☂️ Code Coverage

% Stmts% Branch% Funcs% Lines
94.64 🟢78.26 🟡95.96 🟢94.12 🟢
Click to expand
File% Stmts% Branch% Funcs% Lines
src/manifest-input
index.ts97.79 🟢89.47 🟢100 🟢97.78 🟢
cloneObject.ts100 🟢100 🟢100 🟢100 🟢
dynamicImportWrapper.ts100 🟢100 🟢100 🟢100 🟢
reduceToRecord.ts100 🟢100 🟢100 🟢100 🟢
src
helpers.ts93.55 🟢58.33 🟠88.89 🟢92.31 🟢
manifest.ts100 🟢100 🟢100 🟢100 🟢
index.ts86.84 🟢44.44 🔴100 🟢86.11 🟢
modules.d.ts100 🟢100 🟢100 🟢100 🟢
src/manifest-input/manifest-parser
combine.ts100 🟢100 🟢100 🟢100 🟢
index.ts95.24 🟢75 🟡100 🟢94.87 🟢
permissions.ts100 🟢50 🔴100 🟢100 🟢
validate.ts100 🟢77.78 🟡100 🟢100 🟢
src/html-inputs
index.ts94.55 🟢80.95 🟢100 🟢94.34 🟢
cheerio.ts100 🟢100 🟢100 🟢100 🟢
src/browser-polyfill
index.ts88.89 🟢64.1 🟠66.67 🟡88.57 🟢
src/validate-names
index.ts100 🟢100 🟢100 🟢100 🟢
src/mixed-format
index.ts91.67 🟢50 🔴100 🟢95.65 🟢
regenerateBundle.ts92.31 🟢86.67 🟢100 🟢92.31 🟢
resolveFromBundle.ts92.86 🟢83.33 🟢100 🟢92.31 🟢
src/plugin-reloader-simple
index.ts96.23 🟢83.33 🟢100 🟢95.92 🟢
CONSTANTS.ts100 🟢100 🟢100 🟢100 🟢
src/browser-polyfill/browser
executeScriptPolyfill.ts0 🔴0 🔴0 🔴0 🔴

@jacksteamdev jacksteamdev merged commit e83143c into beta Aug 4, 2021
@jacksteamdev jacksteamdev deleted the fix-schema-bugs branch August 4, 2021 22:26
This was referenced Aug 4, 2021
jacksteamdev added a commit that referenced this pull request Aug 18, 2021
* fix: delay reload by 100ms

* tests: wait for reloader delay
skip delay if delay is 0

* 3.6.0-0

* Add MV3 types and JSON schema (#89)

* Fix integration bugs with new schema (#90)

* fix: add strict schema and fix schema bugs

* feat: upgrade manifest validation errors

* tests: upgrade TS and Jest
fix a bazillion ts-eslint errors
increase jest beforeAll timeout

* fix bug where pkg version could be undefined

* fix: use strict manifest for build validation

* chore: one more time!

* chore: again

* chore: include schema on npm

* chore: support node >= 12

* chore: remove comment from rollup config

* chore: fix types build step

* chore: add alias to jest

* chore: remove aliases

* 3.6.0-1

* chore: fix tsconfig one last time

* Update test fixtures for MV3 (#92)

* tests: rename mv2 parser test
use readJSONSync instead of require

* refactor: rename manifest types file

* chore: rename tests and fix test paths

* tests: create mv3 copies of all crx's

* tests: format test fixture manifest.json files

* chore: add mv2 to all mv3 manifests

* chore: upgrade all fixture manifests to mv3

* chore: ignore script files

* tests: remove junk files from google's conversion script

* tests: make scripts into modules

* tests: rename background file

* tests: rename fixture background files to service_worker

* chore: move and rename background sw files

* tests: remove mv3 irrelevant fixtures

* tests: rename fixture

* tests: side effect

* MV3 Parser (#93)

* tests: rename mv2 parser test
use readJSONSync instead of require

* refactor: rename manifest types file

* chore: rename tests and fix test paths

* tests: create mv3 copies of all crx's

* tests: format test fixture manifest.json files

* chore: add mv2 to all mv3 manifests

* chore: upgrade all fixture manifests to mv3

* chore: ignore script files

* tests: remove junk files from google's conversion script

* tests: make scripts into modules

* tests: rename background file

* tests: rename fixture background files to service_worker

* chore: move and rename background sw files

* tests: remove mv3 irrelevant fixtures

* tests: rename fixture

* feat: parse mv3

* 3.6.0-2

* Support locales (#95)

* tests: add locales fixtures

* feat: support locales in both mv2 and mv3

* fix: tighten filename validation to commonjs chunk

* Remove prettier (#96)

* refactor: remove prettier usage

* chore: make prettier a devDependency

* 3.6.0-3

* Build MV3 extensions (#98)

* 3.6.0-5

* chore: fix build error

* fix: allow ports in schema url match patterns

* 3.6.0-6

* fix: remove debugging line

* fix: warn deprecated feature correctly

* Manifest as ts files (#99)

* refactor deprecation warnings into own file

* refactor manifest v3 update into own file

* tests: add tests for manifest as ts file

* feat: support manifest as TS files

* fix build errors

* 3.6.0-7

* fix deprecation warning condition

* fix: convert content script match patterns (#100)

* fix duplicate match patterns

* fix duplicate map calls

* feat: simplify MV3 content script approach

* 3.6.0-8

* fix manifest v2 schema bug

* MV3 simple reloader (#103)

* refactor: switch to lodash

* feat: add mv3 support to simpleReloader

* fix manifest json schema bug

* refactor simple reloader
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant