Address blimpich review nits for tag GL codes PR

Remove unnecessary comments, restore TagPicker blank line, and use
inline eslint-disable for GL Code naming in the unit test.

Author: neerajbachani

src/components/Search/SearchList/ListItem/TransactionListItem/types.ts

@@ -30,7 +30,6 @@ type TransactionListItemSharedProps<TItem extends ListItem> = {
     exportedReportActions: ReportAction[];
     policyCategories?: PolicyCategories;
     policyTagLists?: PolicyTagLists;
-    /** Policy fields merged from the row snapshot and live Onyx data. */
     rowPolicy?: Policy;
     nonPersonalAndWorkspaceCards?: CardList;
     isAttendeesEnabledForMovingPolicy?: boolean;

src/components/TagPicker/index.tsx

@@ -165,6 +165,7 @@ function TagPicker({
               data: option.data.sort((a, b) => localeCompare(a.text ?? '', b.text ?? '')),
           }))
         : tagSections;
+
     const selectedOptionKey = sections.at(0)?.data?.find((policyTag) => policyTag.searchText === selectedTag)?.keyForList;
 
     const textInputOptions = {

src/components/TransactionItemRow/DataCells/TagCell.tsx

@@ -31,8 +31,6 @@ function TagCell({canEdit, onSave, shouldUseNarrowLayout, shouldShowTooltip, tra
 
     const [livePolicy] = useOnyx(`${ONYXKEYS.COLLECTION.POLICY}${policyID}`);
     const [policyTags] = useOnyx(`${ONYXKEYS.COLLECTION.POLICY_TAGS}${policyID}`);
-    // Search rows can have policy fields that are absent from the live Onyx policy entry. Preserve those
-    // snapshot fields while allowing newer live values (including optimistic toggle updates) to win.
     const policy = livePolicy ? {...policyProp, ...livePolicy} : policyProp;
 
     const policyHasDependentTags = hasDependentTags(policy, policyTags);

src/libs/TagsOptionsListUtils.ts

@@ -102,7 +102,6 @@ function getTagListSections({
 }) {
     const tagSections = [];
     const sortedTags = sortTags(tags, localeCompare);
-    // O(1) lookup for selected/recent shims that lack the PolicyTag GL Code field.
     const tagByName = new Map(sortedTags.map((tag) => [tag.name, tag]));
     const withGLCode = (tag: SelectedTagOption | PolicyTag): TagOptionInput => {
         if (!shouldShowGLCode) {

tests/unit/TagsOptionsListUtilsTest.ts

@@ -959,12 +959,11 @@ describe('TagsOptionsListUtils', () => {
     });
 
     describe('getTagListSections GL code display', () => {
-        /* eslint-disable @typescript-eslint/naming-convention -- PolicyTag GL Code field uses backend naming */
+        // eslint-disable-next-line @typescript-eslint/naming-convention -- PolicyTag GL Code field uses backend naming
         const tagsWithGLCode: Record<string, {name: string; enabled: boolean; 'GL Code'?: string}> = {
-            ProjectA: {name: 'Project A', enabled: true, 'GL Code': 'SP4100'},
+            ProjectA: {name: 'Project A', enabled: true, 'GL Code': 'SP4100'}, // eslint-disable-line @typescript-eslint/naming-convention
             ProjectB: {name: 'Project B', enabled: true},
         };
-        /* eslint-enable @typescript-eslint/naming-convention */
 
         it('sets alternateText when shouldShowGLCode is true and tag has a GL code', () => {
             const result = getTagListSections({