Developer
Daniel Imms
2193314+tyriar@users.noreply.github.com
Performance
YoY:+322%Key patterns and highlights from this developer's activity.
Breakdown of growth, maintenance, and fixes effort over time.
Bugs introduced vs. fixed over time.
Reclassifies engineering effort based on bug attribution. Commits that introduced bugs are retrospectively counted as poor investments.
Investment Quality reclassifies engineering effort based on bug attribution data. Commits identified as buggy origins (those that introduced bugs later fixed by someone) have their grow and maintenance time moved into the Wasted Time category. Their waste (fix commits) remains counted as productive. All other commits retain their standard classification: grow is productive, maintenance is maintenance, and waste (fixes) is productive.
The standard model classifies commits as Growth, Maintenance, or Fixes. Investment Quality adds a quality lens: a commit that introduced a bug is retrospectively counted as a poor investment — the engineering time spent on it was wasted because it ultimately required additional fix work. Fix commits (Fixes in the standard model) are reframed as productive, because fixing bugs is valuable work.
Currently computed client-side from commit and bug attribution data. Ideal server-side endpoint:
POST /v1/organizations/{orgId}/investment-quality
Content-Type: application/json
Request:
{
"startTime": "2025-01-01T00:00:00Z",
"endTime": "2025-12-31T23:59:59Z",
"bucketSize": "BUCKET_SIZE_MONTH",
"groupBy": ["repository_id" | "deliverer_email"]
}
Response:
{
"productivePct": 74,
"maintenancePct": 18,
"wastedPct": 8,
"buckets": [
{
"bucketStart": "2025-01-01T00:00:00Z",
"productive": 4.2,
"maintenance": 1.8,
"wasted": 0.6
}
]
}Latest analyzed commits from this developer.
| Hash | Message | Date | Files |
|---|
Commit activity distribution by hour and day of week. Shows when this developer is most active.
Developers who frequently work on the same files and symbols. Higher score means stronger code collaboration.
| Effort |
|---|
| a31e6e00 | Merge pull request #304498 from microsoft/tyriar/remove | Mar 24 | 0 | – |
| 8657f435 | This commit performs **maintenance** by updating various ownership and attribution details across the codebase. It reassigns responsibilities for **terminal-related modules** and **telemetry classifications** from 'Tyriar' to new owners like 'anthonykim1', 'hediet', and 'meganrogge'. Additionally, it updates **GitHub automation configurations** in `.github/CODENOTIFY` and `.github/classifier.json`, and removes an entry from `.mailmap`. This ensures accurate internal project management and attribution, including updating URL references in `src/vs/platform/terminal/node/terminalProcess.ts` comments. The changes are purely administrative and do not affect any user-facing functionality. | Mar 24 | 12 | maint |
| 0da3fd7f | Merge pull request #295864 from microsoft/tyriar/revert | Feb 17 | 0 | – |
| edb22b0d | This commit **reverts** a previous change that configured the default **terminal tab title** to display a sequence number. It specifically undoes modifications within `src/vs/workbench/contrib/terminal/common/terminalConfiguration.ts`, restoring the default behavior to use the **process name** for terminal tab titles. This **revert** ensures that the terminal's default naming convention for tabs aligns with the prior established behavior. The change primarily affects the **user interface** of the **integrated terminal**, impacting how new terminal tabs are identified by default. | Feb 17 | 1 | maint |
| c5f3a8b4 | Bumps the `ajv` dependency within the `/test/mcp` module. This is an isolated change affecting only the test suite and has no production impact. | Feb 17 | 2 | – |
| efe6157e | Merge pull request #294703 from microsoft/tyriar/osc_notifications | Feb 13 | 0 | – |
| c5a4f26b | Merge branch 'main' into tyriar/osc_notifications | Feb 13 | 0 | – |
| 30cd06b9 | This commit provides a **bug fix** for the **Terminal Notifications** subsystem, specifically addressing an oversight in the notification update logic. It ensures that the `existing.handle` is correctly assigned to `handleRef.current` within the `updateOsc99Notification` function in `src/vs/workbench/contrib/terminalContrib/notification/browser/terminalNotificationHandler.ts`. This **maintenance fix** resolves a recent issue, preventing potential problems with notification state management and ensuring proper handling of OSC 99 notifications in the terminal. The change improves the robustness and reliability of terminal notification updates. | Feb 13 | 1 | waste |
| 92de02a6 | Isolated style change to the VS Code terminal instance; it has no functional impact on users or other modules. | Feb 13 | 1 | maint |
| e17015d6 | This commit implements a **bug fix** within the **terminal notification handler** to **strip markdown links** from **OSC 99 terminal notifications**. It introduces a new internal function, `_sanitizeOsc99MessageText`, which is applied to the title and body of these notifications to prevent raw markdown links from being displayed. A new test case in `terminalNotification.test.ts` verifies that markdown links are correctly removed, ensuring cleaner and more readable terminal output for users. This change improves the presentation of messages originating from the terminal. | Feb 13 | 2 | waste |
| c6636b60 | This commit performs **maintenance refactoring** within the **Terminal Notification Contribution** module to enhance type safety. It specifically modifies the `getValue` call in `src/vs/workbench/contrib/terminalContrib/notification/browser/terminal.notification.contribution.ts` by adding a type parameter. Additionally, it introduces an explicit `=== true` check when evaluating the `isEnabled` symbol, ensuring more robust and predictable type handling for configuration values. This change improves the internal reliability of the **Terminal Notification** feature without altering its external behavior. | Feb 13 | 1 | maint |
| 2d4622e0 | This commit performs **maintenance** by **clarifying the documentation** for the `terminal.integrated.enableNotifications` setting within the **Terminal Notifications** configuration. It updates the description in `src/vs/workbench/contrib/terminalContrib/notification/common/terminalNotificationConfiguration.ts` to provide a more precise understanding of the setting's behavior and any associated limitations. This improves user comprehension of the terminal's notification system without altering its functional behavior. | Feb 13 | 1 | maint |
| 48c90325 | This commit provides a **bug fix** for **terminal OSC 99 notifications**, specifically addressing issues with **custom buttons**. It ensures the correct handling of custom button actions, notification close reasons, and activation reporting within the `src/vs/workbench/contrib/terminalContrib/notification/browser/terminalNotificationHandler.ts` module. This improves the reliability of interactive terminal notifications and is validated by new test cases added to `src/vs/workbench/contrib/terminalContrib/notification/test/browser/terminalNotification.test.ts`. | Feb 13 | 2 | waste |
| 65bfb330 | This commit introduces a **style improvement** to the **terminal notification formatting** within the `formatMessage` function. It modifies the **terminal notification handler** (`terminalNotificationHandler.ts`) to separate the notification title and body with a colon and space (`: `) instead of a newline. This change aims to enhance the readability and consistency of **terminal notifications** presented to the user. Corresponding **unit tests** (`terminalNotification.test.ts`) have been updated to reflect this new expected message structure. | Feb 13 | 2 | maint |
| 03664058 | Merge pull request #295198 from microsoft/tyriar/295196 | Feb 13 | 0 | – |
| 71a9aaa7 | This commit introduces a **critical fix** within the **Terminal Notification Contribution** module, addressing an incorrect method used for writing to the underlying terminal process. Specifically, it **replaces the erroneous `_ctx.processManager.write` call with the correct `_ctx.instance.sendText` method** in the `writeToProcess` context. This **self-correction** ensures that all commands and data are reliably transmitted to the **integrated terminal**, preventing potential communication failures and ensuring proper functionality of terminal notifications. | Feb 13 | 1 | waste |
| 3c0d80a8 | This commit introduces a **security enhancement** to the **Terminal Chat Agent Tools** by **explicitly denying auto-approval** for `git log` commands that include the `--output` flag. This **bug fix**, addressing issue #295196, prevents potential unintended file writes or data leakage by ensuring such commands now require user confirmation before execution. A new configuration entry in `terminalChatAgentToolsConfiguration.ts` enforces this policy, while a corresponding test case in `runInTerminalTool.test.ts` verifies the correct behavior. This change improves user control and safeguards against risky operations when interacting with the terminal via chat agents. | Feb 13 | 2 | waste |
| 75c6849d | Merge remote-tracking branch 'origin/main' into tyriar/osc_notifications | Feb 13 | 0 | – |
| 768af7b8 | This commit performs a significant **maintenance refactoring** within the **Terminal Notifications** subsystem to standardize naming conventions. It renames files and updates import paths to consistently use "notification" instead of "notifications" across the `src/vs/workbench/contrib/terminalContrib/notification` module. Crucially, the internal `Osc99NotificationHandler` class and its file were renamed to `TerminalNotificationHandler` for improved clarity and consistency. This work enhances **code readability** and **maintainability** within the terminal contribution without introducing any functional changes to the user experience. | Feb 13 | 6 | maint |
| e23d2e94 | This commit performs a **maintenance refactoring** of the **Terminal OSC 99 notification** handling within the `terminalContrib` module. It extracts and reorganizes the notification logic into a dedicated `TerminalOscNotificationsContribution` and introduces a new `terminalNotificationsConfiguration.ts` file for managing related settings. The `Osc99NotificationHandler` was refactored to better manage notification actions and disposal. Additionally, the `ITerminalInstance` interface and its implementation were updated to expose a public `isVisible` property, enhancing visibility tracking for terminal instances. This work improves the modularity, maintainability, and configurability of terminal notifications. | Feb 12 | 11 | maint |
Merge pull request #304498 from microsoft/tyriar/remove
This commit performs **maintenance** by updating various ownership and attribution details across the codebase. It reassigns responsibilities for **terminal-related modules** and **telemetry classifications** from 'Tyriar' to new owners like 'anthonykim1', 'hediet', and 'meganrogge'. Additionally, it updates **GitHub automation configurations** in `.github/CODENOTIFY` and `.github/classifier.json`, and removes an entry from `.mailmap`. This ensures accurate internal project management and attribution, including updating URL references in `src/vs/platform/terminal/node/terminalProcess.ts` comments. The changes are purely administrative and do not affect any user-facing functionality.
Merge pull request #295864 from microsoft/tyriar/revert
This commit **reverts** a previous change that configured the default **terminal tab title** to display a sequence number. It specifically undoes modifications within `src/vs/workbench/contrib/terminal/common/terminalConfiguration.ts`, restoring the default behavior to use the **process name** for terminal tab titles. This **revert** ensures that the terminal's default naming convention for tabs aligns with the prior established behavior. The change primarily affects the **user interface** of the **integrated terminal**, impacting how new terminal tabs are identified by default.
Bumps the `ajv` dependency within the `/test/mcp` module. This is an isolated change affecting only the test suite and has no production impact.
Merge pull request #294703 from microsoft/tyriar/osc_notifications
Merge branch 'main' into tyriar/osc_notifications
This commit provides a **bug fix** for the **Terminal Notifications** subsystem, specifically addressing an oversight in the notification update logic. It ensures that the `existing.handle` is correctly assigned to `handleRef.current` within the `updateOsc99Notification` function in `src/vs/workbench/contrib/terminalContrib/notification/browser/terminalNotificationHandler.ts`. This **maintenance fix** resolves a recent issue, preventing potential problems with notification state management and ensuring proper handling of OSC 99 notifications in the terminal. The change improves the robustness and reliability of terminal notification updates.
Isolated style change to the VS Code terminal instance; it has no functional impact on users or other modules.
This commit implements a **bug fix** within the **terminal notification handler** to **strip markdown links** from **OSC 99 terminal notifications**. It introduces a new internal function, `_sanitizeOsc99MessageText`, which is applied to the title and body of these notifications to prevent raw markdown links from being displayed. A new test case in `terminalNotification.test.ts` verifies that markdown links are correctly removed, ensuring cleaner and more readable terminal output for users. This change improves the presentation of messages originating from the terminal.
This commit performs **maintenance refactoring** within the **Terminal Notification Contribution** module to enhance type safety. It specifically modifies the `getValue` call in `src/vs/workbench/contrib/terminalContrib/notification/browser/terminal.notification.contribution.ts` by adding a type parameter. Additionally, it introduces an explicit `=== true` check when evaluating the `isEnabled` symbol, ensuring more robust and predictable type handling for configuration values. This change improves the internal reliability of the **Terminal Notification** feature without altering its external behavior.
This commit performs **maintenance** by **clarifying the documentation** for the `terminal.integrated.enableNotifications` setting within the **Terminal Notifications** configuration. It updates the description in `src/vs/workbench/contrib/terminalContrib/notification/common/terminalNotificationConfiguration.ts` to provide a more precise understanding of the setting's behavior and any associated limitations. This improves user comprehension of the terminal's notification system without altering its functional behavior.
This commit provides a **bug fix** for **terminal OSC 99 notifications**, specifically addressing issues with **custom buttons**. It ensures the correct handling of custom button actions, notification close reasons, and activation reporting within the `src/vs/workbench/contrib/terminalContrib/notification/browser/terminalNotificationHandler.ts` module. This improves the reliability of interactive terminal notifications and is validated by new test cases added to `src/vs/workbench/contrib/terminalContrib/notification/test/browser/terminalNotification.test.ts`.
This commit introduces a **style improvement** to the **terminal notification formatting** within the `formatMessage` function. It modifies the **terminal notification handler** (`terminalNotificationHandler.ts`) to separate the notification title and body with a colon and space (`: `) instead of a newline. This change aims to enhance the readability and consistency of **terminal notifications** presented to the user. Corresponding **unit tests** (`terminalNotification.test.ts`) have been updated to reflect this new expected message structure.
Merge pull request #295198 from microsoft/tyriar/295196
This commit introduces a **critical fix** within the **Terminal Notification Contribution** module, addressing an incorrect method used for writing to the underlying terminal process. Specifically, it **replaces the erroneous `_ctx.processManager.write` call with the correct `_ctx.instance.sendText` method** in the `writeToProcess` context. This **self-correction** ensures that all commands and data are reliably transmitted to the **integrated terminal**, preventing potential communication failures and ensuring proper functionality of terminal notifications.
This commit introduces a **security enhancement** to the **Terminal Chat Agent Tools** by **explicitly denying auto-approval** for `git log` commands that include the `--output` flag. This **bug fix**, addressing issue #295196, prevents potential unintended file writes or data leakage by ensuring such commands now require user confirmation before execution. A new configuration entry in `terminalChatAgentToolsConfiguration.ts` enforces this policy, while a corresponding test case in `runInTerminalTool.test.ts` verifies the correct behavior. This change improves user control and safeguards against risky operations when interacting with the terminal via chat agents.
Merge remote-tracking branch 'origin/main' into tyriar/osc_notifications
This commit performs a significant **maintenance refactoring** within the **Terminal Notifications** subsystem to standardize naming conventions. It renames files and updates import paths to consistently use "notification" instead of "notifications" across the `src/vs/workbench/contrib/terminalContrib/notification` module. Crucially, the internal `Osc99NotificationHandler` class and its file were renamed to `TerminalNotificationHandler` for improved clarity and consistency. This work enhances **code readability** and **maintainability** within the terminal contribution without introducing any functional changes to the user experience.
This commit performs a **maintenance refactoring** of the **Terminal OSC 99 notification** handling within the `terminalContrib` module. It extracts and reorganizes the notification logic into a dedicated `TerminalOscNotificationsContribution` and introduces a new `terminalNotificationsConfiguration.ts` file for managing related settings. The `Osc99NotificationHandler` was refactored to better manage notification actions and disposal. Additionally, the `ITerminalInstance` interface and its implementation were updated to expose a public `isVisible` property, enhancing visibility tracking for terminal instances. This work improves the modularity, maintainability, and configurability of terminal notifications.