Skip to content

Commit b07d7e6

Browse files
Merge pull request #361 from DonOmalVindula/fix/javascript-lint
Fix lint issues in the packages
2 parents 30a8574 + acafcb2 commit b07d7e6

242 files changed

Lines changed: 7849 additions & 7328 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.changeset/ten-aliens-know.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
---
2+
'@asgardeo/browser': patch
3+
'@asgardeo/nextjs': patch
4+
'@asgardeo/react': patch
5+
'@asgardeo/node': patch
6+
---
7+
8+
Fix lint issues in packages

e2e/playwright-report-embedded/index.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,4 +82,4 @@
8282
<div id='root'></div>
8383
</body>
8484
</html>
85-
<script id="playwrightReportBase64" type="application/zip">data:application/zip;base64,UEsDBBQAAAgIANxsTlwut8JYmQYAAAcrAAAZAAAAMTAyOWMzMGM3ZjhiMWIwNmFjZjMuanNvbs1aa2/bNhT9KwS/2AlsWe/X1gJrlqLBgm5o0hVYnQ20RNtcZNEQqaRd6v8+UJIThZFtSZHSGv5gW9Il7zlH9L2HuoNzEuGzEPpQU3UvMNTAmbszbabaKJgbcJQdf49WGPqQkUU8JrHC1jhQOIMjyDHjDPqf77JPO8OMw7kahKYXam5oetjUrNDVxOWER1ngJU2jECQ4DnEC+BIDvJrhMMQh+PmCLOKzGExeg4Cu1jTGMQe3hC/BnCYrMCc4CsVM1gn9Fwe8mGmwTOiKpCs4ghENECc0hv5dlktlHhGJMfTtEQxolK5i6DubEQzTpLhS01V9BFEcU579IlK+GkGOFsUnmvKAZiPjL2sccByKKSG+hP5neLrNRaQyPovB24jewqsRTDBLowI/eTDGUcIvSRZTV3V7rOpjzbxUXV91fUtXbNv8C4oQPPkKfVVcgNcFFQWqb/CcJhi8o/Ra5Hg4oiUiliZiWl5V3LfkC08TDKZwltBbhpMprBXelcKrmlsV/hylcbAERew6kR31SWT7IfLVCCLOUbBc4ZgXPwQ0jTn0tRFk12S9xiH05yhieNPo5FEVJgGNOf7Ca2BiKLZrSzM3qhA5STDiGBSRa8V1pLjad8NjjRa4Hhie93jSpm7tQUPEfXbU/rBoC9x7dEMWIj9OwRROxFJF4hr4mYqmS/ev7hzItMHS6DwsjZq92Z3KCLJYfOfQhwAAG3wDAIj/huHguYv8YAQQ+xoHYHgnuN8cgVevwd00BgA42TAAoFtEeKYMZUE5HQ4K/AZHP2XngeK80uvv7IALvsEyC59EnDlNAMMRDjhNQIYVTYaDzyHiaCxSIuGrKURsgZIQ0/GWqavBUQ2yTF17TJZhqp2Rpant2PIKeCYTcA+A4Oni198GrJokTgGJCScoIv9hgOJ7fkvECYQ19SlF4tNbmlwUEA8vTs9PTy5//3ChbIWRj6iIhQ+RGCcjcMfJCtOU+8BQ/1FVdXOAWU2TmD3N/p/BFHL6Bv9JGJlFeArr0jtOGU5itMJ1eXZsXVqJve5o1kv1il6bZgFJBo3+iJK8cBlmzGzR2MnIFoazeJ3yoyOlBOZwNyNtXzmTRvvprhFjtzQJq6fbpT62I9XWhyvrozt5GO3k0ass+mOxH9GZ7afL0tmK8Dcp5zTuVXP5QPUVZ/amOLOkOK2+4nrVRG8Udiy4XG9WFm8jCeSXOcdJzUZOMCw3cqq9v4+rWaOL0FLDcqBo6at1qpiJplb3ThFl9VsnU3FNKa5p/HDtwtUI4iShSXEe44inDPpQ3AyZ8/DEqZBiiwj0Gvo8SXMq9po3pu7iEAeWi3Xsap6O0cx8at6wNAgwY/M0ir4CsUIAEuc1/A2KSAiCBIc4FpUiAzcEPRT/98VkRy6O5uy0cQzTfjkbJxvsoNw8rVMb5z5iqQ00K9vodred6xnSbbevR29w2+2L+2M6FqbiqZKlpeuVjlYTx0JE9eSozg+3BHXgWFiKZkiZao7euBhRlEn+XuJojRM2QSlfjosvj9zdkr2rmQ08DCvvl4CduxmHTQa5Jsg9Btt5WZPBUgxNai50t3mxVx9fryW+boaXbDrsshsiisIdRgOYJ3SVXXz26x855p5MWbemQzXTjtp61EcdVHlkzSpG7kZCDY0MS7HkBdpubi7WlpKjtpRSr3T3Sut+Qcku1lsSRWAKUbgSq0FvpNuWvGdzoBh+FuklO8uu71dk4OQo6TI/DQ0LZU6iaFgqVO+P7+SnVZcHHKPxTB83uk9nuj0uLRAthdLMxrIU25Ht7D6FYrQUSu/66JXXbhVoNp7pIzNECSISXMtG1on4sRffylIcVd7eau6k15dYycmy6ztZLyCAzojrRk6FmqxdpW2cNweChhoUG7Jx5fRYrzpOuyLDsbf1am7LPRgfD55G4U2MxNYZYJgzwDBjhMZF5RqSBAeciXp2EiK2nFGUhDmUzo4i4+OH8+Hg+Pjh9OPjQYvS1K1fw9yPpNziSPgi7zAKSbx4Vlm61Grd7Z4qPcGgGwce6XiWFNyWUuiGrN5I2S8FUSxvdm1EvEM3+OOH8xp9vK1optQbmJ1tKeilTkCvz4zm5TWhru4y/4WbX+Q4nEynD0xNDjv7GX76np3lk7yZuMxctYbaF3DK2u8OztL/mu7UhlM3Cjgr91KWuQIFoqXEh4NPuUDBDAXXe57AeALs8/dABIaSteTp+03QmqZfReTvtAVSlaNW6T422gGxFe3JE1Mv8vDYd9wBudr8D1BLAwQUAAAICADcbE5cKEwTasgBAABWBAAACwAAAHJlcG9ydC5qc29uxZPBitswEIZfRcxZSS3HTmxTemshl15a6GHJQZHGsRpZMtJ4t0vwuxc5TrPQXXpZqE4zg2b++T+hC/RIUkuS0FxAKhql/eHDGUOERkwcIslA302P0IjdTmTltsyrKq856DFIMt5Bsyvqei3KTX07JYfWWIzQPFzmaK+hAZHltdpkatdWR3HMtlK1G7je/CqTAERzcivj1nFAtaYIHAgjXcek6M0xK91mShe1FpUuaixEqSuR2g3ZeXDnR6tZQKcxMOqQYX9ErVGzj9/Mye0d+/CJKd8P3qEj9mSoY60PPWsNWp02GYL/iYqWTVUXfG/GHjhYrxYOV6+v+rDGITRbDsrbsU/MppcERZ7lHKRznuZKsnzgQPK0RH4k5Wdl/DWgItRpJUkdNA/w+eYlWVntHfti/ROkrjM0FEbkEDCOdiEpiaTqenRzfpgOE/8X3iKvUKMqK8yxEnWO8lj8jTeOSmGM7WjtM0sMmHFXlI/SGs1UQI2OjLSRPRp5f4M/4N+Js9i9CXpTbP8j6MP8oVJ6AfIkLTQ5vyulZHT3NOPQWnl+nqN4NsOwVG96U5r4gljSuTN7dzUOGIIPN1TDQvAyceil6ozDq9HfUEsBAj8DFAAACAgA3GxOXC63wliZBgAABysAABkAAAAAAAAAAAAAALSBAAAAADEwMjljMzBjN2Y4YjFiMDZhY2YzLmpzb25QSwECPwMUAAAICADcbE5cKEwTasgBAABWBAAACwAAAAAAAAAAAAAAtIHQBgAAcmVwb3J0Lmpzb25QSwUGAAAAAAIAAgCAAAAAwQgAAAAA</script>
85+
<script id="playwrightReportBase64" type="application/zip">data:application/zip;base64,UEsDBBQAAAgIACljT1zEP0SwPgcAAIBCAAAZAAAAMTAyOWMzMGM3ZjhiMWIwNmFjZjMuanNvbu1ba0/jRhT9K6P5QkB5+BU/0u5KXQpaVLStFrYrldBqYk/IFNsTecawW8p/ryZxsD3Yie0kEAqID449vp65955r+5zrOzgmPj7x4ACqiua4uuJaY3ukjhQTuWMdtmfHP6EAwwFk5CrskLDLptjtcgbbkGPGGRxc3M22Ss10vLHieobjqbZnONhQ+56titMJ92eGJzT2PRDh0MMR4BMMcDDCnoc98OMZuQpPQtB7D1waTGmIQw5uCZ+AMY0CMCbY98RMphH9G7s8mak7iWhA4gC2oU9dxAkN4eButpbCdfgkxHBgtqFL/TgI4cC6b0MvjpIzbctpQxSGlM92iBVftiFHV8kWjblLZxfG36bY5dgTM0J8AgcX8GixFLGSzkkIjn16Cy/bMMIs9hP3SddiHEX8nMxMaopmdhSto/bPFXPQ7w9Uq6to9h9QWODRdzhQxAl4mgQi8ekHPKYRBh8pvRYrXGlRV4TF7DzUIrPH5BuPIwyGcBTRW4ajIaxkXc9b71tWkfVTFIfuBCSmKxk2ZMNmaviyDRHnyJ0EOOTJDpfGIYcDsbhrMp1iDw7GyGf4vtbgdpFHXBpy/I1X8oipqvmJm0X+OIww4hgkhiuZ1fJmjWfzxhRd4YqusPJz1mxjiTOE3UpWbdmq/hS+aOq4T+iGXIn1cQqGsCdqFAkr+c9RpJirxoqV1qiJVloTVfO+fCltyELxm8MBBMNYUdTRhaMEAJjg3+Sn7gQAAHGjaC32aMHeurV/L7W+2NKDduaSi00zQOx76GaOtO5ELt3vg/TEd+8zx++GYW4plrQUkLV9iwjPHJ1l/4PZbnrkinKaXX8S6cwy9tPzfsgsLj8XkJvLYlNd+ECdzy/9+zM5oGnBI4cpsnE7axxms/QrIlx4HzDsY5fTCMxyiUatvQsPcdQR4SXeuyFE7ApFHqadRSZf7u2vTGa7qxhyMXD6G0tmVVk3m52yFHCUoNcDD+4ReXz28y97rDiJOQUkJJwgn/yDAQof8j+T2GWhV5VNpKEYd0yjsySMrXTQ2dHp0eH5r5/PioCVsbCA6Xx9hUPEfQuREEfFyLzjJMA05unRQUFO9wNdSXf/pSjK/bMjRFVLEXI0ewIEQ8jpB/w7YWTk4yGsCpNOzHAUogBXxYtuSA9AhrU5uGiZB2KtAVzybpJcqNVI4/ljdas0mxfe3UoWL0JyEk5jvr9fOCYT7NZWsrPpX62s1l9KSKaIsVsaeeuFZJO4XcyoMm4d+aFtc7DV14Xtq4bma0PBSyhMxksJCYtHAeEfYs5puBN1aT6h6lVJ31pVMjJVSW1SlV41Ll8ZBJ6nKNWqSX0pICVvJTkw/zTmOKrIyQo09vNodOzllGxFyq3AstKQGluTBy1aYzER6lNWnQe1u0bfzJvVV7AIz0D+XbYhjiIaJeMYRzxmcABF+ZgpCI8UB8m2sECv4YBH8TwSSzUYQ7Oxh92+jTVsq46G0ch4rMGw2HUxY+PY978DUcsBCeec2w3yiQfcCHs4FMwFAzcEpWTdA7mxITFGtcrUGFXT9CeTY+YXW5lu8/vmxvSYB4uZifS15eCvgzpDRt0ywr0G6paY3U31QczZkdnzZVJMJfVhldUdKUAbUB/srmlIOotq1lcfut3e/H+C/SmOWA/FfNJJfuQk2oxGqxqN9Ih+OScFTFmtKH/G2j7BX+uZpxa/b1rPR/DbskKpWSuqxFr54qydL3ZxShQQ/mVUv0+RV0Lyg3FEg9nJJz//Vp6WsuzQIC1fA+G/PcRYsuiyqxHIMWLrRUHtV4rCZspHTeHD6Sq2JBSa9YWPymXEUtYuI28QrgDhN5jtRLErVzePie+DIUReIO74WwO32jdKu6M2ju2Mymk2kUskf8nOlMWUBin9hBpK4ZAx8f1W5s1/qZ1tpO82iTxgycTyrkUozyc3j9DCToX7eEOc11NBna5qqnK3zxaBrq8N9DcsF1W4N/zseoWTxaNdi1BOMyoc4frEva6ilx6KgVuRR52u6sjlqn5vYvVylRFMzSaC6Rswi8vV/xsMT1qG6lWh/mrWMZzz0AIyq+GoyS3+W6WeLWtdDsAqYZfnVOJcG071t1RaSySytugoBgxzBhhmjNAwIRU9EmGXM0E19jzEJiOKIq88/+U29gb5n7xjf/l8mmW6Dw7Syx8crGzMf8FsYAkn/LQ0xaNQ5+1jX4igHzHySHi1XgQ2zARO1Ep3W92WlQJjRZvCWvC214b36wDWW/I/e/nJc8n3Zf2CH9EN/vL5tIKM63RNTe5H3lzrn5Zhz7Um0FKdcoJNq/M5TKYxrqwzLXFaK02A3nCYwq+Xjt1q51qdhNAqfY1yOCf2z2c9IrWLsSl/aLvB9Mi86GhWg/TQ9CXp0aBvcjIvG2UZknFktkh/ndccMELu9ba/5muaKNvuWRR5IrUAFX/LW7tRp8DyM/UsFq1RL+wYqtWz6HRNW/oEX7d3rmVosz2Ll/f/AVBLAwQUAAAICAApY09cjh0weMYBAABLBAAACwAAAHJlcG9ydC5qc29uxZNBi9swEIX/ipizkkaOEzui9NZCLr200MOSgyKNYzWyZKTxbpfg/17kOM1Cd+llobcZoXlv3id0gQ5JGUUK5AWUpkG5HyGeMSaQYuSQSEX6bjsEKapKiHW9EWVZVxzMEBXZ4EFu1sV2udluOTTWYQL5cJmqvQEJYlXs9Hqlq6Y+iuNqq3SzhuvNryrLQrInv7B+mXrUS0rAgTDRVSZXb8osTLPSptwZUZtyh6XYmFrkcUtuEm7D4AyL6A1GRi0y7I5oDBr28Zs9+b1nHz4xHbo+ePTEniy1rAmxY41FZ/ImfQw/UdO8qW5j6OzQAQcX9Jz+mvXVHM56BLnloIMbOg+yGl9yq6sdB+V9oOkgJz5wIHWaqzCQDpMx/upRE5q8kaIW5AN8vkXJSRZ7z7648AR56gyS4oAcIqbBzSAVkdJth37qD+Nh5P+iWxY1GtSbGgusxa5AdSz/ppsGrTGlZnDumWUEzPoryUflrGE6okFPVrnEHq26P8Ef7u+EWVRvcRZFsf6PoA/TL8rtBSiQciALfnfKzeDv7YpD49T5earS2fb9fHrzG7PiC2LZ587s3d04YIwh3lD1M8HLyKFTurUer0F/A1BLAQI/AxQAAAgIACljT1zEP0SwPgcAAIBCAAAZAAAAAAAAAAAAAAC0gQAAAAAxMDI5YzMwYzdmOGIxYjA2YWNmMy5qc29uUEsBAj8DFAAACAgAKWNPXI4dMHjGAQAASwQAAAsAAAAAAAAAAAAAALSBdQcAAHJlcG9ydC5qc29uUEsFBgAAAAACAAIAgAAAAGQJAAAAAA==</script>

package.json

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,11 @@
6969
"seroval": "1.4.1",
7070
"qs": "6.14.1",
7171
"@vitejs/plugin-vue>vite": "7.1.12",
72-
"prettier": "2.6.2"
72+
"prettier": "2.6.2",
73+
"@wso2/eslint-plugin>@typescript-eslint/eslint-plugin": "6.21.0",
74+
"@wso2/eslint-plugin>@typescript-eslint/parser": "6.21.0",
75+
"@wso2/eslint-plugin>@typescript-eslint/type-utils": "6.21.0",
76+
"@wso2/eslint-plugin>@typescript-eslint/utils": "6.21.0"
7377
}
7478
},
7579
"publishConfig": {

packages/browser/.eslintignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
/dist
22
/build
33
/node_modules
4-
/coverage
4+
/coverage
5+
/src/__legacy__

packages/browser/esbuild.config.mjs

Lines changed: 34 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -16,88 +16,89 @@
1616
* under the License.
1717
*/
1818

19-
import { readFileSync } from 'fs';
19+
import {readFileSync} from 'fs';
20+
import {createRequire} from 'module';
2021
import * as esbuild from 'esbuild';
21-
import { createRequire } from 'module';
2222
import inlineWorkerPlugin from 'esbuild-plugin-inline-worker';
2323

2424
const require = createRequire(import.meta.url);
2525
const pkg = JSON.parse(readFileSync('./package.json', 'utf8'));
2626

2727
// Get dependencies excluding crypto-related ones that need to be bundled
28-
const externalDeps = [...Object.keys(pkg.dependencies || {}), ...Object.keys(pkg.peerDependencies || {})]
29-
.filter(dep => !['crypto-browserify', 'randombytes', 'buffer'].includes(dep));
28+
const externalDeps = [...Object.keys(pkg.dependencies || {}), ...Object.keys(pkg.peerDependencies || {})].filter(
29+
dep => !['crypto-browserify', 'randombytes', 'buffer'].includes(dep),
30+
);
3031

3132
// Plugin to alias crypto and buffer modules
3233
const polyfillPlugin = {
3334
name: 'polyfill-plugin',
3435
setup(build) {
3536
// Crypto polyfill
36-
build.onResolve({ filter: /^crypto$/ }, () => ({
37-
path: require.resolve('crypto-browserify')
37+
build.onResolve({filter: /^crypto$/}, () => ({
38+
path: require.resolve('crypto-browserify'),
3839
}));
3940

4041
// Buffer polyfill
41-
build.onResolve({ filter: /^buffer$/ }, () => ({
42-
path: require.resolve('buffer/')
42+
build.onResolve({filter: /^buffer$/}, () => ({
43+
path: require.resolve('buffer/'),
4344
}));
44-
}
45+
},
4546
};
4647

4748
const commonOptions = {
48-
bundle: true,
49-
entryPoints: ['src/index.ts'],
50-
external: externalDeps,
51-
platform: 'browser',
52-
target: ['es2020'],
53-
define: {
54-
global: 'globalThis', // Required by crypto-browserify
55-
'process.env.NODE_DEBUG': 'false',
56-
'process.version': '"16.0.0"',
57-
'process.browser': 'true'
58-
},
5949
banner: {
6050
js: `
6151
import { Buffer } from 'buffer/';
6252
if (typeof window !== 'undefined' && !window.Buffer) {
6353
window.Buffer = Buffer;
6454
}
65-
`
55+
`,
56+
},
57+
bundle: true,
58+
define: {
59+
global: 'globalThis', // Required by crypto-browserify
60+
'process.browser': 'true',
61+
'process.env.NODE_DEBUG': 'false',
62+
'process.version': '"16.0.0"',
6663
},
64+
entryPoints: ['src/index.ts'],
65+
external: externalDeps,
6766
footer: {
6867
js: `
6968
if (typeof window !== 'undefined' && !window.Buffer) {
7069
window.Buffer = require('buffer/').Buffer;
7170
}
72-
`
71+
`,
7372
},
73+
platform: 'browser',
7474
plugins: [
7575
polyfillPlugin,
7676
inlineWorkerPlugin({
77-
format: 'iife',
78-
target: 'es2020',
79-
platform: 'browser',
8077
define: {
81-
'global': 'self',
82-
'globalThis': 'self',
78+
global: 'self',
79+
globalThis: 'self',
80+
'process.browser': 'true',
8381
'process.env.NODE_DEBUG': 'false',
8482
'process.version': '"16.0.0"',
85-
'process.browser': 'true'
86-
}
87-
})
88-
]
83+
},
84+
format: 'iife',
85+
platform: 'browser',
86+
target: 'es2020',
87+
}),
88+
],
89+
target: ['es2020'],
8990
};
9091

9192
await esbuild.build({
9293
...commonOptions,
9394
format: 'esm',
9495
outfile: 'dist/index.js',
95-
sourcemap: true
96+
sourcemap: true,
9697
});
9798

9899
await esbuild.build({
99100
...commonOptions,
100101
format: 'cjs',
101102
outfile: 'dist/cjs/index.js',
102-
sourcemap: true
103+
sourcemap: true,
103104
});

packages/browser/src/__legacy__/client.ts

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ import {
2626
IdToken,
2727
OIDCEndpoints,
2828
User,
29+
createPackageComponentLogger,
2930
} from '@asgardeo/javascript';
3031
import WorkerFile from '../web.worker';
3132
import {MainThreadClient, WebWorkerClient} from './clients';
@@ -47,6 +48,11 @@ import {
4748
import {BrowserStorage} from './models/storage';
4849
import {SPAUtils} from './utils';
4950

51+
const logger: ReturnType<typeof createPackageComponentLogger> = createPackageComponentLogger(
52+
'@asgardeo/browser',
53+
'AsgardeoSPAClient',
54+
);
55+
5056
/**
5157
* Default configurations.
5258
*/
@@ -85,15 +91,15 @@ export class AsgardeoSPAClient {
8591
this._instanceID = id;
8692
}
8793

88-
public instantiateAuthHelper(authHelper?: typeof AuthenticationHelper) {
94+
public instantiateAuthHelper(authHelper?: typeof AuthenticationHelper): void {
8995
if (authHelper) {
9096
this._authHelper = authHelper;
9197
} else {
9298
this._authHelper = AuthenticationHelper;
9399
}
94100
}
95101

96-
public instantiateWorker(worker: new () => Worker) {
102+
public instantiateWorker(worker: new () => Worker): void {
97103
if (worker) {
98104
this._worker = worker;
99105
} else {
@@ -123,8 +129,7 @@ export class AsgardeoSPAClient {
123129

124130
while (!this._initialized) {
125131
if (iterationToWait === 1e4) {
126-
// eslint-disable-next-line no-console
127-
console.warn('It is taking longer than usual for the object to be initialized');
132+
logger.warn('It is taking longer than usual for the object to be initialized');
128133
}
129134
await sleep();
130135
iterationToWait++;

packages/browser/src/__legacy__/http-client/helpers/decorators.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,14 +17,11 @@
1717
*
1818
*/
1919

20-
/* eslint-disable @typescript-eslint/no-empty-function */
21-
/* eslint-disable @typescript-eslint/no-unused-vars */
22-
2320
/**
2421
* A decorator to supplement static interface support.
2522
*
2623
* @return {<U extends T>(constructor: U) => void}
2724
*/
2825
export function staticDecorator<T>() {
29-
return <U extends T>(_constructor: U): any => {};
26+
return <U extends T>(constructor: U): U => constructor;
3027
}

packages/browser/src/index.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,13 @@
1919
/**
2020
* Entry point for all public APIs of this SDK.
2121
*/
22+
// eslint-disable-next-line import/no-cycle
2223
export * from './__legacy__/client';
24+
// eslint-disable-next-line import/no-cycle
2325
export * from './__legacy__/models';
2426

2527
// Utils
28+
// eslint-disable-next-line import/no-cycle
2629
export * from './__legacy__/utils/spa-utils';
2730

2831
// Constants

packages/browser/src/theme/themeDetection.ts

Lines changed: 14 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,12 @@
1616
* under the License.
1717
*/
1818

19-
import {ThemeDetection, ThemeMode} from '@asgardeo/javascript';
19+
import {ThemeDetection, ThemeMode, createPackageComponentLogger} from '@asgardeo/javascript';
20+
21+
const logger: ReturnType<typeof createPackageComponentLogger> = createPackageComponentLogger(
22+
'@asgardeo/browser',
23+
'ThemeDetection',
24+
);
2025

2126
/**
2227
* Extended theme detection config that includes DOM-specific options
@@ -51,11 +56,11 @@ export const detectThemeMode = (mode: ThemeMode, config: BrowserThemeDetection =
5156

5257
if (mode === 'class') {
5358
if (!targetElement) {
54-
console.warn('ThemeDetection: targetElement is required for class-based detection, falling back to light mode');
59+
logger.warn('ThemeDetection: targetElement is required for class-based detection, falling back to light mode');
5560
return 'light';
5661
}
5762

58-
const classList = targetElement.classList;
63+
const {classList} = targetElement;
5964

6065
// Check for explicit dark class first
6166
if (classList.contains(darkClass)) {
@@ -84,10 +89,10 @@ export const createClassObserver = (
8489
): MutationObserver => {
8590
const {darkClass = 'dark', lightClass = 'light'} = config;
8691

87-
const observer = new MutationObserver(mutations => {
88-
mutations.forEach(mutation => {
92+
const observer: MutationObserver = new MutationObserver((mutations: MutationRecord[]) => {
93+
mutations.forEach((mutation: MutationRecord) => {
8994
if (mutation.type === 'attributes' && mutation.attributeName === 'class') {
90-
const classList = targetElement.classList;
95+
const {classList} = targetElement;
9196

9297
if (classList.contains(darkClass)) {
9398
callback(true);
@@ -101,8 +106,8 @@ export const createClassObserver = (
101106
});
102107

103108
observer.observe(targetElement, {
104-
attributes: true,
105109
attributeFilter: ['class'],
110+
attributes: true,
106111
});
107112

108113
return observer;
@@ -116,9 +121,9 @@ export const createMediaQueryListener = (callback: (isDark: boolean) => void): M
116121
return null;
117122
}
118123

119-
const mediaQuery = window.matchMedia('(prefers-color-scheme: dark)');
124+
const mediaQuery: MediaQueryList = window.matchMedia('(prefers-color-scheme: dark)');
120125

121-
const handleChange = (e: MediaQueryListEvent) => {
126+
const handleChange = (e: MediaQueryListEvent): void => {
122127
callback(e.matches);
123128
};
124129

packages/browser/src/types/worker.d.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,35 +18,35 @@
1818

1919
// Type declarations for worker files handled by esbuild-plugin-inline-worker
2020

21-
declare module "*.worker" {
21+
declare module '*.worker' {
2222
const WorkerFactory: {
2323
new (): Worker;
2424
};
2525
export default WorkerFactory;
2626
}
2727

28-
declare module "*.worker.js" {
28+
declare module '*.worker.js' {
2929
const WorkerFactory: {
3030
new (): Worker;
3131
};
3232
export default WorkerFactory;
3333
}
3434

35-
declare module "*.worker.ts" {
35+
declare module '*.worker.ts' {
3636
const WorkerFactory: {
3737
new (): Worker;
3838
};
3939
export default WorkerFactory;
4040
}
4141

42-
declare module "*.worker.jsx" {
42+
declare module '*.worker.jsx' {
4343
const WorkerFactory: {
4444
new (): Worker;
4545
};
4646
export default WorkerFactory;
4747
}
4848

49-
declare module "*.worker.tsx" {
49+
declare module '*.worker.tsx' {
5050
const WorkerFactory: {
5151
new (): Worker;
5252
};

0 commit comments

Comments
 (0)