Revert "Improve error handling and remove mutation observer"

[avikalpg]

Reverts Alokit-Innovations/chrome-extension#78

Summary by CodeRabbit

• Refactor: Improved the handling of data storage and retrieval in backgroundScript.js and api.js for better performance and reliability.
• New Feature: Enhanced init.js to initialize the script after the window is loaded, improving the user experience by ensuring the script runs at the right time.
• Documentation: Added a detailed description to the orchestrator.js function, making it easier for developers to understand its purpose and functionality.
• Refactor: Simplified the destroyElement function in utilities.js for better readability and maintenance.
• Chore: Removed redundant setStorage and getStorage functions from utilities.js, streamlining the codebase.

These changes aim to enhance the software's performance, reliability, and maintainability, providing a smoother user experience.

[coderabbitai[bot]]

Walkthrough

The changes primarily focus on improving error handling, code readability, and maintainability. The code now uses promises for asynchronous operations, and the chrome.storage.local.get method to retrieve data from local storage. The orchestrator function has been refactored to receive input parameters directly, and the setStorage and getStorage functions have been removed. The changes also include a more detailed description of the orchestrator function's purpose and functionality.

Changes

File	Summary
backgroundScript.js	The code now uses promises for asynchronous operations and includes additional .then calls to handle the response from the backend API.
scripts/api.js	The function apiCallOnprem has been refactored to include a try-catch block for error handling and uses chrome.storage.local.get to retrieve the token.
scripts/init.js	The script now initializes once the window is loaded, retrieves data from local storage, and sets up a MutationObserver to monitor changes in the document's body.
scripts/orchestrator.js	The orchestrator function has been refactored to receive input parameters directly and includes a detailed description of its functionality.
scripts/utilities.js	The destroyElement function has been simplified, and the setStorage and getStorage functions have been removed.

---

Tips

### Chat with CodeRabbit Bot (`@coderabbitai`) - Mention `@coderabbitai` in any *review comment* for bot assistance. - Note: Review comments are made on code diffs or files, not on the PR overview. ### Pause Incremental Reviews - Insert `@coderabbitai: ignore` in the PR description to halt the bot's ongoing reviews. Remove the line to resume. - Additionally, enter `@coderabbitai pause` to halt the bot's ongoing reviews as any issue or review comment. To resume reviews, enter `@coderabbitai resume` as any issue or review comment.

[vibinex-code-review[bot]]

Relevant users for this PR:	Contributor Name/Alias	Code Coverage
tapishr	99.25%
ajayVibinex	0.75%

Auto-assigning to following reviewers: tapishr ajayVibinex

Code coverage is calculated based on the git blame information of the PR. To know more, hit us up at contact@vibinex.com. To change comment and auto-assign settings, go to your Vibinex settings page.

[sonarcloud[bot]]

SonarCloud Quality Gate failed.   

1 Bug
0 Vulnerabilities
0 Security Hotspots
3 Code Smells

No Coverage information
0.0% Duplication

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

[vibinex-code-review[bot]]

Relevant users for this PR:	Contributor Name/Alias	Code Coverage
tapishr	99.25%
ajayVibinex	0.75%

Auto-assigning to following reviewers: tapishr ajayVibinex

Code coverage is calculated based on the git blame information of the PR. To know more, hit us up at contact@vibinex.com. To change comment and auto-assign settings, go to your Vibinex settings page.