add more tests, use the Sourcegraph stubs api and improve repo matching.

README.md

@@ -1,7 +1,8 @@
-[![codecov](https://codecov.io/gh/sourcegraph/sentry/branch/master/graph/badge.svg)](https://codecov.io/gh/sourcegraph/sourcegraph-sentry)
-
 # Sentry extension
 
+[![build](https://travis-ci.org/sourcegraph/sentry.svg?branch=master)](https://travis-ci.org/sourcegraph/sentry)
+[![codecov](https://codecov.io/gh/sourcegraph/sentry/branch/master/graph/badge.svg)](https://codecov.io/gh/sourcegraph/sourcegraph-sentry)
+
 Sentry helps devs track, organize and break down errors more efficiently, facilitating their debug process. We want to make it more convenient for developers to access Sentry's error tracking tools directly from the code that is doing the error handling, code such as `throw new Error(QUERY)`, `console.log(QUERY)`, `console.error(QUERY)` etc..
 
 The Sentry extension renders a `View logs in Sentry` next to error throwing statements, linking directly to the corresponding Sentry issues stream page. Links are rendered when viewing files on [Sourcegraph](https://sourcegraph.com), GitHub and GitLab.
@@ -34,24 +35,31 @@ Set the following configurations in your settings:
   {
     "name": "[Project name for the config overview, e.g. Webapp errors]",
     "projectId": "[Sentry project ID, e.g. "1334031"]",
-    "patternProperties": {
-      "repoMatches": "[RegExp[] repo names asociated with this Sentry project]",
-      "fileMatches": [
-          [RegExp[] that matches file format, e.g. "\\.tsx?"]
-        ],
-      "lineMatches": [
-        [RegExp[] that matches error handling code, e.g. "throw new Error+\\(['\"]([^'\"]+)['\"]\\)"],
+    "linePatterns": [
+        // List of RegExp patterns that match error handling code, e.g. "throw new Error+\\(['\"]([^'\"]+)['\"]\\)",
+        // !! Make sure to capture the error message in a RegExp group !!
       ]
+    "filters": {
+        [
+            "repositories": [
+                // List of RegExp repo names asociated with this Sentry project
+            ],
+            "files": [
+                // List of RegExp that matches file format, e.g. "\\.tsx?",
+                // or for more specific matching, folder matching, e.g. "(?:web|shared|src)\/.*\\.tsx?"
+        ],
     }
   }
 
 ```
 
-File matches can also be narrowed down to certain folders by specifying this in the RegExp:
+## Important features:
+
+File patterns can also be narrowed down to certain folders by specifying this in the RegExp:
 
 ```
 ...
-"fileMatches": ["(web|shared|src)\/.*\\.tsx?"]
+"files": ["(?:web|shared|src)\/.*\\.tsx?"]
 ...
 ```
 
@@ -62,25 +70,60 @@ File matches can also be narrowed down to certain folders by specifying this in
   Configuration:
 
   ```
-  ...
-  "patternProperties": {
-      "repoMatches": "sourcegraph",
-      "fileMatches": ["([^'\"]+)\/.*\\.ts?"],
-      "lineMatches": [
-          "throw new Error+\\(['\"]([^'\"]+)['\"]\\)",
-          "console\\.(warn|debug|info|error)\\(['\"`]([^'\"`]+)['\"`]\\)"
-          ]
+  "sentry.decorations.inline": true,
+  "sentry.organization": "sourcegraph",
+  "sentry.projects": [
+    {
+        "name": "sourcegraph",
+        "projectId": "1334031",
+        "linePatterns": [
+            "throw new Error+\\(['\"]([^'\"]+)['\"]\\)",
+            "console\\.(warn|debug|info|error)\\(['\"`]([^'\"`]+)['\"`]\\)"
+            ]
+        "filters": [
+            {
+                "repositories": "sourcegraph\/sourcegraph",
+                "files": ["web\/.*\\.ts?"],
+            },
+            {
+                "files": ["sourcegraph-subfolder\/.*\\.tsx?"]
+            }
+
+        ]
     }
+  ]
 
   ```
 
-  - [On Sourcegraph](https://sourcegraph.com/github.com/sourcegraph/sourcegraph/-/blob/client/browser/src/libs/github/file_info.ts#L16)
-  - [On GitHub](https://github.com/sourcegraph/sourcegraph/blob/master/client/browser/src/libs/github/file_info.ts#L16)
+  - [On Sourcegraph](https://sourcegraph.com/github.com/sourcegraph/sourcegraph/-/blob/browser/src/libs/github/file_info.ts#L22)
+  - [On GitHub](https://github.com/sourcegraph/sourcegraph/blob/master/browser/src/libs/github/file_info.ts#L22)
 
 - Go
 
-  - [On Sourcegraph](https://sourcegraph.com/github.com/sourcegraph/sourcegraph/-/blob/cmd/frontend/auth/user_test.go#L54:19)
-  - [On GitHub](https://github.com/sourcegraph/sourcegraph/blob/master/cmd/frontend/auth/user_test.go#L54)
+Configuration:
+
+```
+"sentry.decorations.inline": true,
+"sentry.organization": "sourcegraph",
+"sentry.projects": [
+  "name": "Dev env errors",
+  "projectId": "213332",
+  "linePatterns": ["errors\\.New\\(['\"`](.*)['\"`]\\)"],
+  "filters": [
+      {
+          "repositories": ["sourcegraph\/sourcegraph", "sourcegraph\/dev-repo"],
+          "files": ["/auth\/.*.go?/"],
+      },
+      {
+          "repositories": ["/dev-env/"]
+      }
+  ],
+]
+
+```
+
+- [On Sourcegraph](https://sourcegraph.com/github.com/sourcegraph/sourcegraph/-/blob/cmd/frontend/auth/user_test.go#L54:19)
+- [On GitHub](https://github.com/sourcegraph/sourcegraph/blob/master/cmd/frontend/auth/user_test.go#L54)
 
 - JavaScript
 

package.json

@@ -6,7 +6,7 @@
   "activationEvents": [
     "*"
   ],
-  "wip": true,
+  "wip": false,
   "categories": [
     "External services"
   ],
@@ -44,7 +44,6 @@
           "alt": "sentry.link.fileList",
           "when": "resource && !config.sentry.hideSentryButton"
         }
-
       ],
       "commandPalette": [
         {
@@ -89,22 +88,22 @@
               "repoMatches": {
                 "type": "array",
                 "items": {
-                    "type": "string"
-                  },
+                  "type": "string"
+                },
                 "description": "Regex to match repos associated to this Sentry project, e.g. github\\.com/sourcegraph/sourcegraph"
               },
               "fileMatches": {
                 "type": "array",
                 "items": {
-                    "type": "string"
-                  },
+                  "type": "string"
+                },
                 "description": "Regex to match files associated with this project, e.g. (web|shared)/.*\\.tsx?$"
               },
               "lineMatches": {
                 "type": "array",
                 "items": {
-                    "type": "string"
-                  },
+                  "type": "string"
+                },
                 "description": "Regex to match lines associated with this project, e.g. throw new Error\\([\"']([^'\"]+)[\"']\\)"
               }
             },
@@ -175,9 +174,10 @@
     "last 1 Safari versions"
   ],
   "devDependencies": {
+    "@sourcegraph/extension-api-stubs": "^0.2.0",
     "@sourcegraph/prettierrc": "^2.2.0",
     "@sourcegraph/tsconfig": "^4.0.0",
-    "@sourcegraph/tslint-config": "^13.0.0",
+    "@sourcegraph/tslint-config": "^13.1.0",
     "@types/expect": "1.20.4",
     "@types/lodash": "4.14.123",
     "@types/mocha": "5.2.6",
@@ -188,17 +188,17 @@
     "lnfs-cli": "^2.1.0",
     "lodash": "^4.17.11",
     "mkdirp": "^0.5.1",
-    "mocha": "^6.1.2",
+    "mocha": "^6.1.4",
     "mock-require": "^3.0.3",
-    "nyc": "^13.3.0",
+    "nyc": "^14.1.0",
     "parcel-bundler": "^1.12.3",
     "prettier": "1.17.0",
-    "rxjs": "^6.4.0",
-    "sinon": "^7.3.1",
+    "rxjs": "^6.5.1",
+    "sinon": "^7.3.2",
     "source-map-support": "^0.5.12",
-    "sourcegraph": "^23.0.0",
-    "ts-node": "^8.0.3",
-    "tslint": "^5.15.0",
-    "typescript": "^3.4.2"
+    "sourcegraph": "^23.0.1",
+    "ts-node": "^8.1.0",
+    "tslint": "^5.16.0",
+    "typescript": "^3.4.5"
   }
 }