Feature: Save settings in local storage

[bdougie]

Type of feature

🍕 Feature

Current behavior

I use the same settings every-time I generate a description. But every-time there is an update or the cookie refreshes I need to save the settings again. It would be nice if we save the user settings in local storage.

Suggested solution

localstorage is preferred, but we could potentially store a settings JSON.

"settings" : {
  "Description Source" : "Commit Message",
  "Description Type" : "Formal"
}

Additional context

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

Contributing Docs

• I agree to follow this project's Contribution Docs

[Anush008]

We are saving the settings to chrome.storage.local for it to be persistent.

ai/src/popup/pages/aiprdescription.tsx

Lines 35 to 39 in cc76e95

	const handleFormSubmit = (e: React.FormEvent<HTMLFormElement>) => {
	e.preventDefault();
	void setAIDescriptionConfig(config);
	toast.success("Configuration updated!");
	};

ai/src/utils/ai-utils/descriptionconfig.ts

Lines 39 to 41 in cc76e95

	export const setAIDescriptionConfig = async (data: DescriptionConfig): Promise<void> => {
	await chrome.storage.local.set({ [AI_PR_DESCRIPTION_CONFIG_KEY]: data });
	};

The problem being, the storage is reset if there's a reinstall.

[nickytonline]

@Anush008, consider using chrome.storage.sync instead. See https://developer.chrome.com/docs/extensions/reference/storage/#storage-areas. The only caveat is it can't store much, only 100KB but it will perist across all devices and shouldn't get wiped between installs (need to double check that)

Also, see https://pandaquests.medium.com/data-retention-made-easy-persist-data-with-chrome-storage-api-in-chrome-extensions-ea4f4c721a62

[Anush008]

Right. The extension settings are being saved to the synced storage.

ai/src/popup/pages/settings.tsx

Lines 15 to 30 in f67f673

	useEffect(() => {
	const getSettingsDataFromStorage = async () => {
	const settingsConfig = await chrome.storage.sync.get("osSettingsConfig");
	if (settingsConfig.osSettingsConfig === undefined) {
	const defaultSettings = { aiPrDescription: true, codeRefactor: true };
	await chrome.storage.sync.set({ osSettingsConfig: defaultSettings });
	setSettingsConfig(defaultSettings);
	} else {
	setSettingsConfig(settingsConfig.osSettingsConfig);
	}
	};
	void getSettingsDataFromStorage();
	}, []);

We can do same for the AI description settings too.

[a0m0rajab]

@Anush008 do we have anything to do for this issue?