Extension fails with Slack

[pconerly]

Hi Daniel!

I'm getting a funny bug when I use slack. The stacktrace is:

Error in event handler for (unknown): ReferenceError: windo is not defined
    at eval (eval at <anonymous> (chrome-extension://hojgjeiaognigdpphinpoobgpmgpkone/jquery-1.11.1.min.js:2:2620), <anonymous>:9:2)
    at chrome-extension://hojgjeiaognigdpphinpoobgpmgpkone/jquery-1.11.1.min.js:2:2620
    at Function.globalEval (chrome-extension://hojgjeiaognigdpphinpoobgpmgpkone/jquery-1.11.1.min.js:2:2631)
    at m.fn.init.domManip (chrome-extension://hojgjeiaognigdpphinpoobgpmgpkone/jquery-1.11.1.min.js:3:22994)
    at m.fn.init.append (chrome-extension://hojgjeiaognigdpphinpoobgpmgpkone/jquery-1.11.1.min.js:3:20507)
    at m.fn.init.<anonymous> (chrome-extension://hojgjeiaognigdpphinpoobgpmgpkone/jquery-1.11.1.min.js:3:22038)
    at m.access (chrome-extension://hojgjeiaognigdpphinpoobgpmgpkone/jquery-1.11.1.min.js:3:3286)
    at m.fn.init.html (chrome-extension://hojgjeiaognigdpphinpoobgpmgpkone/jquery-1.11.1.min.js:3:21623)
    at chrome-extension://hojgjeiaognigdpphinpoobgpmgpkone/context-script.js:30:19
rollup-core_required_libs.423f98a82fc5aba43877.min.js:1 

I imagin that CutSpel is reriting javascript, which it thinks is text, and specifically is turning window into windo.

Let me know if there's any more debug info I can provide you with!

[DanielTillett]

Yes I would say that too. I have not ever used cutspel with slack. The code checks if nodeType === 3 which is supposed to be text only. I wonder if slack is doing something non-standard.

[pconerly]

Would checking for <script> element tags help? Slack has this element in the body:

<script>
	
	setTimeout(function() {
		if (documnt.getElementById('self_help_link_div')) {
			documnt.getElementById('self_help_link_div').styl.opacity = 1;
		}
	}, 30000);
	// Only log 10% of al events
	if (windo.TSMeasureAndBeacon) {
		windo.TSMeasureAndBeacon('loading_screen_rendered', 'start_nav');
	}
	
</script>

And there's also an error on documnt when the setTimeout fires

[DanielTillett]

I would need to look into this in more depth, but you could fix this easily on your end by removing “window” and “document” from the cutspel.csv list.