Last-mile typecheck sweep: 3 small bugs

- GuideToolOngoingView classNames clause: classNames(..., 'chat.roomId'
  && 'cursor-pointer') — the property name was quoted so the literal
  string 'chat.roomId' was always-truthy. Unquote to read the actual
  chat.roomId field.
- NavigatorRoomSettingsModTabView: UserProfileIconView userName={ user.userId }
  put a number into the string-typed userName prop; the right prop for
  a numeric id is userId.
- WiredExtraVariableEchoView resolvedVariableEntries: the inline
  fallback-entry literal at the bottom of the useMemo got its kind
  field widened to string (instead of the 'custom' literal needed by
  IWiredVariablePickerEntry). Lift it into a typed const + rename to
  namedFallback to avoid the shadowing of the upstream
  createFallbackVariableEntry result.

src/components/wired/views/extras/WiredExtraVariableEchoView.tsx

@@ -129,7 +129,7 @@ export const WiredExtraVariableEchoView: FC<{}> = () =>
         if(fallbackEntry) return [ fallbackEntry, ...variableEntries ];
         if(!fallbackSourceName) return variableEntries;
 
-        return [ {
+        const namedFallback: IWiredVariablePickerEntry = {
             id: sourceVariableToken,
             token: sourceVariableToken,
             label: fallbackSourceName,
@@ -139,7 +139,9 @@ export const WiredExtraVariableEchoView: FC<{}> = () =>
             hasValue: true,
             kind: 'custom',
             target: sourceTargetType
-        }, ...variableEntries ];
+        };
+
+        return [ namedFallback, ...variableEntries ];
     }, [ fallbackSourceName, sourceTargetType, sourceVariableToken, variableEntries ]);
 
     const selectedEntry = useMemo(() => flattenWiredVariablePickerEntries(resolvedVariableEntries).find(entry => (entry.token === sourceVariableToken)) ?? null, [ resolvedVariableEntries, sourceVariableToken ]);