🚇 Release 1.0.1

.github/workflows/publish.yml

@@ -0,0 +1,34 @@
+name: Publish to VS Code Marketplace
+on:
+  push:
+    branches: [main]
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Setup Node version
+        uses: actions/setup-node@v1
+        with:
+          node-version: 16
+
+      - uses: actions/checkout@v2
+
+      - name: Make envfile
+        uses: SpicyPizza/[email protected]
+        with:
+          envkey_SESSIONS_SECRET_KEY: ${{ secrets.SESSION_SECRET_KEY }}
+          envkey_CLIENT_IDENTIFIER: com.pullflow.vscode
+          envkey_PULLFLOW_APP_URL: https://app.pullflow.com
+
+      - name: Build, compile and test
+        run: |
+          yarn
+          yarn compile
+          yarn lint
+          yarn test
+
+      - name: publish
+        run: |
+          yarn global add vsce
+          vsce publish -p ${{ secrets.VSCE_ACCESS_TOKEN }}

.vscode/launch.json

@@ -3,7 +3,7 @@
 // Hover to view descriptions of existing attributes.
 // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
 {
-	"version": "1.0.0",
+	"version": "1.0.1",
 	"configurations": [
 		{
 			"name": "Run Extension",

.vscode/settings.json

@@ -16,10 +16,12 @@
         "dbaeumer",
         "nosources",
         "Pullflow",
+        "quickpicks",
         "Relogin",
         "thumbsup",
         "uuidv",
         "vscodeignore",
+        "Webviews",
         "Xids"
     ],
     "editor.tabSize": 2,

package.json

@@ -2,8 +2,8 @@
   "name": "pullflow",
   "publisher": "Pullflow",
   "displayName": "Pullflow",
-  "description": "Enhance your code review process by consolidating interactions within VS Code and directly integrating with platforms like Slack and GitHub.",
-  "version": "1.0.0",
+  "description": "Code review collaboration across GitHub, Slack, and VS Code.",
+  "version": "1.0.1",
   "preview": true,
   "license": "MIT",
   "engines": {
@@ -14,11 +14,14 @@
   },
   "icon": "assets/pullflow.png",
   "categories": [
-    "SCM Providers",
+    "Programming Languages",
+    "Machine Learning",
+    "Education",
+    "Snippets",
     "Other"
   ],
   "galleryBanner": {
-    "color": "#795DBD",
+    "color": "#F6F4F9",
     "theme": "light"
   },
   "private": true,
@@ -42,11 +45,11 @@
       },
       {
         "command": "pullflow.login",
-        "title": "Pullflow: login"
+        "title": "Pullflow: Login"
       },
       {
         "command": "pullflow.logout",
-        "title": "Pullflow: logout"
+        "title": "Pullflow: Logout"
       },
       {
         "command": "pullflow.reconnect",
@@ -59,6 +62,10 @@
       {
         "command": "pullflow.toggle-flow-state",
         "title": "Pullflow: Toggle Flow state"
+      },
+      {
+        "command": "pullflow.welcome-view",
+        "title": "Pullflow: Welcome"
       }
     ],
     "keybindings": [

src/commands/reconnect.ts

@@ -47,5 +47,8 @@ export const Reconnect = async (
     userAuthoredCodeReviews: codeReviews.data.authored,
   })
   StatusBar.update({ context, statusBar, state: StatusBarState.SignedIn })
+
+  commands.executeCommand(Command.activePullRequests)
+
   return
 }

src/commands/welcomeView.ts

@@ -0,0 +1,6 @@
+import { ExtensionContext } from 'vscode'
+import { Welcome } from '../views/webviews/welcome/welcome'
+
+export const WelcomeView = (context: ExtensionContext) => {
+  Welcome.show(context)
+}

src/extension.ts

@@ -10,13 +10,15 @@ import { Logout } from './commands/logout'
 import { ActivePullRequests } from './commands/activePullRequests'
 import { Authorization } from './utils/authorization'
 import { ToggleFlowState } from './commands/toggleFlowState'
+import { Welcome } from './views/webviews/welcome/welcome'
 
 const module = 'extension.ts'
 
 export async function activate(context: ExtensionContext) {
   log.info('activating extension', module)
 
-  await checkSessionAndLogin(context)
+  checkFirstActivation(context)
+
   const statusBar: StatusBarItem = await StatusBar.activate(context)
   const { pollIntervalId, focusStateEvent, presenceInterval } =
     await initialize({
@@ -70,17 +72,22 @@ export async function activate(context: ExtensionContext) {
       ToggleFlowState(context, statusBar)
     )
   )
+  context.subscriptions.push(
+    commands.registerCommand(Command.welcomeView, () => {
+      Welcome.show(context)
+    })
+  )
 }
 
 // this method is called when your extension is deactivated
 export function deactivate() {}
 
-const checkSessionAndLogin = async (context: ExtensionContext) => {
-  const session = await Authorization.currentSession(context)
-  if (!session) {
-    window.showInformationMessage(
-      `Pullflow: Please login to continue using extension`
-    )
-    commands.executeCommand(Command.login)
+const checkFirstActivation = (context: ExtensionContext) => {
+  const isFirstActivation = !Store.get(context)?.extensionId
+
+  if (isFirstActivation) {
+    commands.executeCommand(Command.welcomeView)
+    const extensionId = `${context.extension.packageJSON.publisher}.${context.extension.packageJSON.name}`
+    Store.set(context, { extensionId })
   }
 }

src/utils/commands.ts

@@ -4,4 +4,5 @@ export const Command = {
   logout: 'pullflow.logout',
   reconnect: 'pullflow.reconnect',
   toggleFlowState: 'pullflow.toggle-flow-state',
+  welcomeView: 'pullflow.welcome-view',
 }

src/utils/pullRequestsState.ts

@@ -52,9 +52,6 @@ export const PullRequestState = {
       if (errorCount.count >= MAX_ERROR_COUNT) {
         StatusBar.update({ context, statusBar, state: StatusBarState.Error })
       }
-      window.showErrorMessage(
-        `Pullflow: Couldn't fetch pull requests. ${codeReviews.error.message}`
-      )
       return
     }
     await Store.set(context, {

src/utils/types.ts

@@ -38,6 +38,7 @@ export type RepoChannelConnection = {
 }
 
 export type CacheObject = {
+  extensionId?: string
   pendingUserCodeReviews?: [UserCodeReview]
   userAuthoredCodeReviews?: [UserCodeReview]
   user?: User

src/views/statusBar/statusBar.ts

@@ -68,11 +68,15 @@ export const StatusBar = {
     log.info('updating status bar', module)
 
     if (state === StatusBarState.SignedIn) {
-      statusBarProperties.signedIn.text = getStatusBarText(context)
+      statusBarProperties.signedIn.text = getStatusBarText({ context })
       statusBar = Object.assign(statusBar, statusBarProperties.signedIn)
     } else if (state === StatusBarState.Loading) {
       statusBar = Object.assign(statusBar, statusBarProperties.loading)
     } else if (state === StatusBarState.Error) {
+      statusBarProperties.error.text = getStatusBarText({
+        context,
+        showErrorIcon: true,
+      })
       statusBar = Object.assign(statusBar, statusBarProperties.error)
     } else {
       statusBar = Object.assign(statusBar, statusBarProperties.signedOut)
@@ -82,7 +86,13 @@ export const StatusBar = {
 }
 
 /** computes status bar text */
-function getStatusBarText(context: ExtensionContext): string {
+function getStatusBarText({
+  context,
+  showErrorIcon = false,
+}: {
+  context: ExtensionContext
+  showErrorIcon?: boolean
+}): string {
   if (STATUS_BAR_ICON_PREVIEW) {
     const allIcons = IconReference.map((k) => `$(${k})`).join('')
     return '$(pullflow-icon)' + allIcons
@@ -93,12 +103,15 @@ function getStatusBarText(context: ExtensionContext): string {
     previousPresenceStatus,
   } = Store.get(context)
 
-  if (!pendingUserCodeReviews && !userAuthoredCodeReviews) return ''
+  const presenceIcon =
+    previousPresenceStatus === PresenceStatus.Flow ? '$(symbol-event)' : ''
+  const errorIcon = showErrorIcon ? '  $(warning)  ' : ''
+
+  if (!pendingUserCodeReviews && !userAuthoredCodeReviews)
+    return `${errorIcon} ${presenceIcon} $(pullflow-icon)`
 
   const pendingCodeReviewsCount = pendingUserCodeReviews?.length || 0
   const authoredCodeReviewsCount = userAuthoredCodeReviews?.length || 0
-  const presenceIcon =
-    previousPresenceStatus === PresenceStatus.Flow ? '$(symbol-event)' : ''
   if (
     pendingCodeReviewsCount < MAX_PR_COUNT &&
     authoredCodeReviewsCount < MAX_PR_COUNT
@@ -111,10 +124,10 @@ function getStatusBarText(context: ExtensionContext): string {
       userAuthoredCodeReviews?.map(
         (codeReview) => `$(${PullRequestIcons.getIcon(codeReview.botReaction)})`
       ) || []
-    return `${presenceIcon} ${pendingCodeReviewIcons.join(
+    return `${errorIcon} ${presenceIcon} ${pendingCodeReviewIcons.join(
       ''
-    )} $(pullflow-icon)\t${authoredCodeReviewIcons.join('')}`
+    )} $(pullflow-icon) ${authoredCodeReviewIcons.join('')}`
   } else {
-    return `${presenceIcon} ${pendingCodeReviewsCount} $(pullflow-icon) ${authoredCodeReviewsCount}`
+    return `${errorIcon} ${presenceIcon} ${pendingCodeReviewsCount} $(pullflow-icon) ${authoredCodeReviewsCount}`
   }
 }