update adr

Author: cemms1

docs/architecture/proposals/004-bundle-refactoring.md

@@ -22,17 +22,27 @@ await insertAds();
 await updateAds();
 ```
 
-### Initialise step
+### Initialise
 
 The consented-advertising file contains lots of information about init scripts that is hard to follow.
 
+This should include what is currently known as `commercialFeatures` and should change `commercialFeatures` from a class into a JSON object or similar.
+
 Open questions:
 
 - How do we know what the correct order/priority is for these init scripts?
 - What is comscore? Why does it need to run first? Why does it run on ad free?
 -
 
-### Coding Standards
+### insertAds
+
+TODO
+
+### updateAds
+
+TODO
+
+## Coding Standards
 
 We propose establishing the following coding standards:
 
@@ -50,4 +60,5 @@ We propose establishing the following coding standards:
     import { initGoogleTag } from '../googletag';
     ```
 
--
+- **Use pure functions** and avoid using classes where possible. The commercial bundle does not really need to use classes
+  When side effects are necessary (like DOM manipulation), clearly note this in a JSDoc comment. Ideally we would include an eslint rule to prevent functions with side effects from being added without a suitable explanation for why this is happening