SSG AMP hybrid mode dynamic route adds "amp":1 to context on build in non AMP article.json

Bug report

Describe the bug

When using a dynamic route in hybrid mode with getStaticProps and getStaticPaths, all static generated files have added amp: 1 to context.

E.g if getStaticPaths returns /article for [slug].js then these files are generated:

article.amp.html
article.amp.json
article.html
article.json

and in article.json context is

context: { 
  params: {
    slug : ["article"],
    amp: 1
  }
}

So first visit on article loads AMP component . Everything goes back to normal once revalidated.

To Reproduce

https://github.com/SuperdeskWebPublisher/publisher-pwa

• download repository.
• build
• go to .next/server/pages/business/WHATEVER.json and look for context (value of context.params is added to pageProps)

Expected behavior

Article.json context should not contain amp: 1

System information

• OS: macOS
• Version of Next.js: 9.5.5
• Version of Node.js: 10.15.0

Additional context

this may be connected: #17245

[tomaszrondio]

I believe that sooner or later there will be a case when someone will really need to know if isAmp at the level of getStaticProps. But yeah so far so good 😃

[balazsorban44]

This issue has been automatically locked due to no recent activity. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.