Currently
questIdFrames[questId:number] = table<frameName:string, frameName:string>
And always when a frame is really needed we do local icon = _G[frameName].

Should rather be:
questIdFrames[questId:number] = table<frameName:string, frameRefence:Frame>
Then we could use the frames without going thru massive global variable table. And we still have name of the frame available.