Here's my initial attempt at #22.

Feel free to pull in and then add to, extend, or clean up by extracting check functions now that there's two methods. There's two TODOs--c and purescript--I don't really plan to get to those.

Notes

• This executes only after no file extensions were matched (extensions still take precedence)
• I included all supported script languages and left out the declarative ones (Markdown, JSON, dependency files, etc), since they have nothing to run them AFAIK
• It only checks for #! and common program names to keep it flexible (since not everyone will use #!/usr/bin/env)
• As a sort of naive conflict resolution, the if statements are sorted by program name length
• This still defaults to markdown like it did before

References

• https://stackoverflow.com/questions/41322300/how-to-execute-rust-code-directly-on-unix-systems-using-the-shebang
• https://github.com/nim-lang/Nim/issues/3301
• https://github.com/crystal-lang/crystal/issues/546
• https://stackoverflow.com/questions/7707178/whats-the-appropriate-go-shebang-line
• https://stackoverflow.com/questions/25880705/when-writing-a-haskell-script-get-syntax-error-near-unexpected-token
• https://stackoverflow.com/questions/29718123/cabal-vs-runhaskell-when-to-use
• https://guide.elm-lang.org/install.html

I created better support for PHP. This fixes #46.

Unfortunately, I wasn't able to link use statements. This could maybe be done with some external API, so if I find (or create) it, I will create another PR.

Changes in this PR related to PHP:

• Support for parsing PHP extensions (link ext-* dependency to documentation) in compsoer.json
• Support for linking name and bin fields in compsoer.json
• Support for linking PHP's include and require

Other changes:

• Allow multiple spaces between key and colon in JSON
• Merge similar code in JSON parser

Can you add better support for PHP?

This should include better composer.json handling.

The current version only links normal dependencies. But it should also link extension dependencies (ext-*) to correct documentation page on PHP documentation website. Suggested dependencies in suggest section should also be linked correctly.

It should also link name field to package on Packagist (but don't then forget to fix #45), autoload and autoload-dev sections to correct directories and bin section to correct file.

Composer's composer.json file is similar to Node's package.json so you could take some code from there. You should also look to composer.json schema for reference.

There should also be parsing of PHP files.

First, it should link require, require_once, include and include_once to correct files. This is similar to Node, except that it can be used with or without brackets.

It should also link use statements. This is a bit different because it includes namespace instead of the file. But with autoloading, program then includes correct file. In this case, this extension should somehow "extract" correct file from namespace and link it.

In https://github.com/sb89/crystal-swapi/blob/master/src/swapi.cr

links to https://crystal-lang.org/api/0.21.1/HTTP.html instead of https://crystal-lang.org/api/0.21.1/HTTP/Client.html

Adapting the work from https://github.com/fiatjaf/gh-browser/blob/master/python.js to a PHP should not be hard for anyone familiar with PHP include and require workings.

Hello!

I have some troubles with this extension, when pressing Back and Forward browser buttons :cold_sweat:

Steps to reproduce:

1. Load some file, application for example — there are highlights.
2. Click to some require (require_relative 'router', for example) — works good, now you in router file with highlights.
3. Press browser Back button — works good, now you in application file again with highlights.
4. Press browser Forward button — BAM, you in router file without highlights (and some dynamic GitHub loading seen).

Thanks.

It looks like the 'don't run more than once' protection is broken because Github changed their markup?

This line injects an element to stop the content script from running multiple times but I think it doesn't work because the element it injects into does not exist (or maybe it doesn't always exist).

In this screenshot, notice that there are multiple dots next to the imports, and the <span id="module-linker-done"> doesn't exist in the page.

See: https://github.com/pypa/setuptools_scm/blob/master/setup.py#L2

important note

in a docstring turns into

important note:ant●pan class="pl-s">important note:

(which is undesired)

Related issue: #38

I don't know how this works, but I see a variety of different languages, so I guess you have some flexible solution. Would it be possible to add Scala support?

Actually, if there were some contribution docs, I could try to add it myself..

When local module is written like below, module linker links remote module having same namesake.

const hello = require('hello');  // it's a local module. not remote.

I suggest that module-linker should find the package.json first. So it can resolve whether the module is local or remote.

e.g. https://raw.githubusercontent.com/fiatjaf/module-linker/backends/data/hackage-modules/Data.Align this should return semialign, but returns these

https://hackage.haskell.org/package/semialign-1.2/docs/Data-Align.html

The extension disappeared from Chrome and from its web store: https://chrome.google.com/webstore/detail/dglofghjinifeolcpjfjmfdnnbaanggn

Any idea why? :)

EDIT: I see the following. Why does chrome say it violates the policy?

Hello!

SASS has partials, which filenames start with _, but it's not written in @import: https://sass-lang.com/documentation/at-rules/import

So, we're having incorrect links (with 404).

Example: https://github.com/nana-4/materia-theme/blob/8988c72/src/chrome/chrome-scrollbar-dark/scrollbars.scss

Incorrect link to https://github.com/nana-4/materia-theme/blob/8988c72/src/_sass/colors.scss

Correct link is https://github.com/nana-4/materia-theme/blob/8988c72/src/_sass/_colors.scss

Hey there, great extension!

I can't really say why but just wanted to point out that there might a memory leak. Been running my mac book for 5 days, 46 mins and the memory has reached 1.6 gb. Cheers

Error in event handler for runtime.onMessage: TypeError: Cannot read property '1' of null
    at main (chrome-extension://dglofghjinifeolcpjfjmfdnnbaanggn/dist/full.js:22:64)
    at chrome-extension://dglofghjinifeolcpjfjmfdnnbaanggn/dist/full.js:118:3

Error occurs at this page: https://github.com/getsentry/sentry/pull/7521/files

Sometimes Back and Forward in browser are broken (I don't know if it cause of this).

import_plugins turns into import_plugins_plugins●ort_plugins. It might be multiline-only bug, but I haven't checked.

Example: https://github.com/ansible/ansible/blob/b9b8081/test/runner/lib/cloud/init.py#L19

source:

from lib.util import (
    ApplicationError,
    display,
    is_shippable,
    import_plugins,
    load_plugins,
    ABC,
)

result:

from lib●.util import (
    ApplicationError,
    display,
    is_shippable,
    import_plugins_plugins●ort_plugins,
    load_plugins,
    ABC,
)