Drop TODOs that are not relevant anymore (#4280)

<!-- PR description-->

---

#### Does this PR need a docs update or release note?

- [ ] ✅ Yes, it's included
- [ ] 🕐 Yes, but in a later PR
- [x] ⛔ No

#### Type of change

<!--- Please check the type of change your PR introduces: --->
- [ ] 🌻 Feature
- [ ] 🐛 Bugfix
- [ ] 🗺️ Documentation
- [ ] 🤖 Supportability/Tests
- [ ] 💻 CI/Deployment
- [x] 🧹 Tech Debt/Cleanup

#### Issue(s)

<!-- Can reference multiple issues. Use one of the following "magic words" - "closes, fixes" to auto-close the Github issue. -->
* #<issue>

#### Test Plan

<!-- How will this be tested prior to merging.-->
- [ ] 💪 Manual
- [ ] ⚡ Unit test
- [ ] 💚 E2E

src/cli/utils/groups.go

@@ -138,8 +138,6 @@ func ValidateGroupsRestoreFlags(backupID string, opts GroupsOpts) error {
 		return clues.New("invalid time format for " + flags.MessageLastReplyBeforeFN)
 	}
 
-	// TODO(meain): selectors (refer sharepoint)
-
 	return validateRestoreConfigFlags(flags.CollisionsFV, opts.RestoreCfg)
 }
 

src/internal/archive/zip.go

@@ -15,7 +15,6 @@ import (
 const (
 	// ZipCopyBufferSize is the size of the copy buffer for zip
 	// write operations
-	// TODO(meain): tweak this value
 	ZipCopyBufferSize = 5 * 1024 * 1024
 )
 

src/internal/m365/collection/drive/collection.go

@@ -198,7 +198,6 @@ func (oc *Collection) Remove(itemID string) bool {
 }
 
 // IsEmpty check if a collection does not contain any items
-// TODO(meain): Should we just have function that returns driveItems?
 func (oc *Collection) IsEmpty() bool {
 	return len(oc.driveItems) == 0
 }

src/internal/m365/collection/drive/collections_test.go

@@ -2492,7 +2492,7 @@ func delItem(
 }
 
 func getDeltaError() error {
-	syncStateNotFound := "SyncStateNotFound" // TODO(meain): export graph.errCodeSyncStateNotFound
+	syncStateNotFound := "SyncStateNotFound"
 	me := odataerrors.NewMainError()
 	me.SetCode(&syncStateNotFound)
 

src/internal/m365/export.go

@@ -33,7 +33,7 @@ func (ctrl *Controller) ProduceExportCollections(
 	defer end()
 
 	ctx = graph.BindRateLimiterConfig(ctx, graph.LimiterCfg{Service: sels.PathService()})
-	ctx = clues.Add(ctx, "export_config", exportCfg) // TODO(meain): needs PII control
+	ctx = clues.Add(ctx, "export_config", exportCfg)
 
 	var (
 		expCollections []export.Collectioner

src/internal/operations/export.go

@@ -34,7 +34,6 @@ import (
 const (
 	// CopyBufferSize is the size of the copy buffer for disk
 	// write operations
-	// TODO(meain): tweak this value
 	CopyBufferSize = 5 * 1024 * 1024
 )