Skip to content

Refactor content script logging & status bar to reduce noise and execution overhead #682

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.

Sign up for GitHub

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Add comprehensive tests and documentation for enhanced logging & stat…

8d8549a
Select commit
Loading
Failed to load commit list.
Open

Refactor content script logging & status bar to reduce noise and execution overhead #682

Add comprehensive tests and documentation for enhanced logging & stat…
8d8549a
Select commit
Loading
Failed to load commit list.
Codacy Production / Codacy Static Code Analysis required action Sep 3, 2025 in 0s

352 new issues (0 max.) of at least severity.

Codacy Here is an overview of what got changed by this pull request:

Issues
======
- Added 352
           

Complexity increasing per file
==============================
- apps/acf-extension/src/content_scripts/batch.ts  3
- apps/acf-extension/src/content_scripts/status-bar.ts  18
- packages/shared/status-bar/src/lib/status-bar.ts  47
- packages/core/common/src/lib/utilities/logger.spec.ts  14
- packages/core/common/src/lib/utilities/logger.ts  16
- apps/acf-extension/src/content_scripts/logger.ts  19
- packages/shared/status-bar/src/lib/status-bar.spec.ts  48
- demo-enhancements.js  9
- apps/acf-extension/src/content_scripts/logger.spec.ts  25
         

Complexity decreasing per file
==============================
+ apps/acf-extension/src/content_scripts/index.ts  -4
         

Clones added
============
- packages/shared/status-bar/src/lib/status-bar.spec.ts  12

See the complete overview on Codacy

Annotations

Check warning on line 22 in apps/acf-extension/src/content_scripts/index.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/index.ts#L22 

ES2017 async function declarations are forbidden.

Check warning on line 27 in apps/acf-extension/src/content_scripts/index.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/index.ts#L27 

Unsafe assignment of an `any` value.

Check warning on line 1 in apps/acf-extension/src/content_scripts/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/logger.ts#L1 

Can't resolve '@dhruv-techapps/acf-store' in '/src/apps/acf-extension/src/content_scripts'

Check warning on line 2 in apps/acf-extension/src/content_scripts/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/logger.ts#L2 

Can't resolve '@dhruv-techapps/core-common' in '/src/apps/acf-extension/src/content_scripts'

Check warning on line 2 in apps/acf-extension/src/content_scripts/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/logger.ts#L2 

Unable to resolve path to module '@dhruv-techapps/core-common'.

Check warning on line 26 in apps/acf-extension/src/content_scripts/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/logger.ts#L26 

Unsafe call of an `error` type typed value.

Check warning on line 26 in apps/acf-extension/src/content_scripts/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/logger.ts#L26 

Unsafe member access .getSettings on an `error` typed value.

Check warning on line 27 in apps/acf-extension/src/content_scripts/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/logger.ts#L27 

ES2015 block-scoped variables are forbidden.

Check warning on line 27 in apps/acf-extension/src/content_scripts/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/logger.ts#L27 

Unsafe member access .logging on an `error` typed value.

Check warning on line 34 in apps/acf-extension/src/content_scripts/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/logger.ts#L34 

Unsafe assignment of an error typed value.

Check warning on line 46 in apps/acf-extension/src/content_scripts/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/logger.ts#L46 

Unsafe member access .TRACE on an `error` typed value.

Check warning on line 7 in apps/acf-extension/src/content_scripts/status-bar.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/status-bar.ts#L7 

ES2022 field 'initialized' is forbidden.

Check warning on line 31 in apps/acf-extension/src/content_scripts/status-bar.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/status-bar.ts#L31 

Unsafe assignment of an error typed value.

Check warning on line 37 in apps/acf-extension/src/content_scripts/status-bar.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/status-bar.ts#L37 

ES2020 nullish coalescing operators are forbidden.

Check warning on line 38 in apps/acf-extension/src/content_scripts/status-bar.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apps/acf-extension/src/content_scripts/status-bar.ts#L38 

Unsafe member access .enableStatusBar on an `error` typed value.

Check warning on line 17 in packages/core/common/src/lib/utilities/logger.spec.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

packages/core/common/src/lib/utilities/logger.spec.ts#L17 

ES2015 arrow function expressions are forbidden.

Check warning on line 49 in packages/core/common/src/lib/utilities/logger.spec.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

packages/core/common/src/lib/utilities/logger.spec.ts#L49 

ES2015 block-scoped variables are forbidden.

Check warning on line 80 in packages/core/common/src/lib/utilities/logger.spec.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

packages/core/common/src/lib/utilities/logger.spec.ts#L80 

ES2015 block-scoped variables are forbidden.

Check warning on line 98 in packages/core/common/src/lib/utilities/logger.spec.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

packages/core/common/src/lib/utilities/logger.spec.ts#L98 

ES2015 block-scoped variables are forbidden.

Check warning on line 23 in packages/core/common/src/lib/utilities/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

packages/core/common/src/lib/utilities/logger.ts#L23 

'ERROR' is defined but never used.

Check warning on line 45 in packages/core/common/src/lib/utilities/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

packages/core/common/src/lib/utilities/logger.ts#L45 

ES2015 modules are forbidden.

Check warning on line 61 in packages/core/common/src/lib/utilities/logger.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

packages/core/common/src/lib/utilities/logger.ts#L61 

ES2015 spread elements are forbidden.

Check warning on line 109 in packages/shared/status-bar/src/lib/status-bar.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

packages/shared/status-bar/src/lib/status-bar.ts#L109 

ES2015 arrow function expressions are forbidden.

Check warning on line 220 in packages/shared/status-bar/src/lib/status-bar.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

packages/shared/status-bar/src/lib/status-bar.ts#L220 

ES2017 async function declarations are forbidden.

Check warning on line 221 in packages/shared/status-bar/src/lib/status-bar.ts

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

packages/shared/status-bar/src/lib/status-bar.ts#L221 

Unsafe assignment of an error typed value.
View more details on Codacy Production