Regression with @extends and @media

[paulfalgout]

I have a feeling this won't be a popular bug, but it seems like a different edge case from #2009 and #2007

I wasn't seeing this in libsass v3.3.2

I think I've isolated the problem to this:

@mixin mq($points, $directions: min, $dimensions: width) {
    $query: '';
    @for $i from 1 through length($points) {
        $query: $query + ' and (#{nth($directions, $i)}-#{nth($dimensions, $i)}: #{nth($points, $i)})';
    }

    @media screen #{$query} {
        @content;
}
}

%color {
    color: #F00;
}


.foo {
    @extend %color;
    padding: 10px;

    @include mq($bp-medium, min, width) {
        padding: 30px;
    }
}

and getting "You may not @extend an outer selector from within @media"

[mgreter]

Your sample (once I added the missing pieces) compiles correctly!
Closing since no version was stated, so I guess it was not the latest!

[paulfalgout]

Ah the sourcemap inspector will be useful. I guess I need to trace down the issue more.

With 0 code changes I did a fresh install of gulp-sass ~2.2.0 which has a dependency on node-sass of ^3.4.1 and node-sass 3.5.x recently updated libsass such that I am now using 3.3.5

If I force gulp-sass to use node-sass 3.4.1 which uses libsass 3.3.2 this error goes away

[paulfalgout]

So perhaps I'm running into a legitimate error that wasn't thrown in the earlier versions. Upon closer inspection of the error I was looking at the wrong line trying to make an example: Here's what breaks:
example @mgreter is this specifically what the error is addressing or is this a regression?

With libsass 3.3.2 it would output with no error:

.screening__item.highlight label {
  box-shadow: inset 0 0 0 1px #0582DA;
}

[mgreter]

Now take that snippet and enter into http://www.sassmeister.com/ to see what official ruby sass thinks about the code snippet (hint: the same as libsass does now). So this was fixed in libsass recently.