Skip to content

Conversation

tmctl
Copy link
Contributor

@tmctl tmctl commented Aug 19, 2025

WIP

tmctl added 3 commits August 7, 2025 16:51
- Enhanced logger with global log level configuration via env/localStorage
- Replaced console.log statements with structured logger throughout codebase
- Added createLogger factory function for module-specific logger instances
- Removed emoji characters from log messages for cleaner output
- Applied prettier formatting to ensure consistent code style

Signed-off-by: Tyler McDonald <[email protected]>
@swirlds-automation
Copy link

swirlds-automation commented Aug 19, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

Copy link

github-actions bot commented Aug 19, 2025

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
88.64% (-1.51% 🔻)
1217/1373
🟢 Branches
81.6% (-3.03% 🔻)
417/511
🟢 Functions
89.27% (-0.79% 🔻)
283/317
🟢 Lines
88.92% (-1.6% 🔻)
1156/1300
Show files with reduced coverage 🔻
St.
File Statements Branches Functions Lines
🟡
... / logger.ts
67.74% (-32.26% 🔻)
70.37% (-29.63% 🔻)
80% (-20% 🔻)
67.74% (-32.26% 🔻)
🟢 reown/adapter.ts
99.25% (-0.75% 🔻)
88.57% (+0.24% 🔼)
100%
99.23% (-0.77% 🔻)
🟢
... / HederaConnector.ts
94.44% (-5.56% 🔻)
100%
80% (-20% 🔻)
94.44% (-5.56% 🔻)
🟢
... / account.ts
80% (-20% 🔻)
75% (-25% 🔻)
100%
80% (-20% 🔻)
🟢
... / HederaProvider.ts
93.25% (-6.75% 🔻)
88.24% (-11.76% 🔻)
100%
93.17% (-6.83% 🔻)

Test suite run success

433 tests passing in 51 suites.

Report generated by 🧪jest coverage report action from 0df0028

tmctl added 2 commits August 19, 2025 16:36
- Test namespace structure consistency between DAppConnector (v1) and HederaProvider/HederaConnector (v2)
- Verify method and event alignment across both versions
- Test required vs optional namespace behavior differences
- Validate URI parameter encoding compatibility
- All 49 tests passing, confirming namespace content equivalence despite different parameter modes

Signed-off-by: Tyler McDonald <[email protected]>
- Add support for reading stored connection params from sessionStorage
- Allow HederaConnector and HederaProvider to use dApp-specified namespaces
- Set internal namespace properties before calling parent connect method
- Replace console.log statements with library logger throughout
- Apply prettier formatting to maintain code consistency

Signed-off-by: Tyler McDonald <[email protected]>
omar-hgraph and others added 7 commits September 10, 2025 09:52
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Tyler McDonald <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Tyler McDonald <[email protected]>
Signed-off-by: Tyler McDonald <[email protected]>
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.

4 participants