feat(tracker): Gantt — #9 inline filter chips + advanced search + view-option toggles

foundations/server/packages/elastic/src/adapter.ts

@@ -314,17 +314,46 @@ class ElasticAdapter implements FullTextAdapter {
     from: number | undefined
   ): Promise<IndexedDoc[]> {
     if (query.$search === undefined) return []
+    const raw = String(query.$search)
+    // Route field-targeted queries (e.g. `searchTitle:value`, `identifier:HULY-`,
+    // `comments.message:foo`) through `query_string` so ES can parse the
+    // field-targeted clauses and apply per-field boosts. Restrict the
+    // detector to the set of fields we actually index — typing a bare colon
+    // such as `POC: design review` or a URL must NOT silently route to
+    // `query_string` (which would throw a parsing exception and surface as
+    // zero hits). Anything else falls back to `simple_query_string` for full
+    // backwards compatibility.
+    const KNOWN_FIELD_RE =
+      /(^|\s)(searchTitle|searchShortTitle|identifier|description\.plain|comments\.message|fulltextSummary)\s*:/i
+    const usesQueryString = KNOWN_FIELD_RE.test(raw)
+    const queryBlock: any = usesQueryString
+      ? {
+          query_string: {
+            query: raw,
+            fields: [
+              'searchTitle^3',
+              'searchShortTitle^2',
+              'identifier^2',
+              'description.plain',
+              'comments.message^0.7',
+              'fulltextSummary'
+            ],
+            default_operator: 'AND',
+            allow_leading_wildcard: false
+          }
+        }
+      : {
+          simple_query_string: {
+            query: raw,
+            analyze_wildcard: true,
+            flags: 'OR|PREFIX|PHRASE|FUZZY|NOT|ESCAPE',
+            default_operator: 'and'
+          }
+        }
     const request: any = {
       bool: {
         must: [
-          {
-            simple_query_string: {
-              query: query.$search,
-              analyze_wildcard: true,
-              flags: 'OR|PREFIX|PHRASE|FUZZY|NOT|ESCAPE',
-              default_operator: 'and'
-            }
-          },
+          queryBlock,
           {
             term: {
               workspaceId

models/tracker/src/index.ts

@@ -38,6 +38,7 @@ import {
   DOMAIN_TRACKER,
   TClassicProjectTypeData,
   TComponent,
+  TDependencyShiftedNotification,
   TIssue,
   TIssueRelation,
   TIssueStatus,
@@ -161,6 +162,34 @@ function defineNotifications (builder: Builder): void {
     tracker.ids.AssigneeNotification
   )
 
+  //  — Notification on Dependency-Shift.
+  // The cascade bundle is created client-side as a CommonInboxNotification
+  // subclass (see `dependency-shift-send.ts`); this NotificationType wires it
+  // into the user's tracker notification group so settings/Inbox provider
+  // routing work the same as the assignee type. `field` is bound to the
+  // `cascadeToken` attribute so the auto-`dueDate` generated type stays
+  // distinct (and they don't compete on the same `dueDate` notify channel).
+  builder.createDoc(
+    notification.class.NotificationType,
+    core.space.Model,
+    {
+      hidden: false,
+      generated: false,
+      label: tracker.string.DependencyShifted,
+      group: tracker.ids.TrackerNotificationGroup,
+      field: 'cascadeToken',
+      txClasses: [core.class.TxCreateDoc],
+      objectClass: tracker.class.DependencyShiftedNotification,
+      templates: {
+        textTemplate: '{sender} shifted {trigger} — {count} dependent issues moved',
+        htmlTemplate: '<p>{sender} shifted {trigger} — {count} dependent issues moved</p>',
+        subjectTemplate: 'Dependency shift'
+      },
+      defaultEnabled: true
+    },
+    tracker.ids.DependencyShiftedNotification
+  )
+
   generateClassNotificationTypes(
     builder,
     tracker.class.Issue,
@@ -196,7 +225,9 @@ function defineFilters (builder: Builder): void {
         key: 'milestone',
         component: view.component.ObjectFilter,
         showNested: false
-      }
+      },
+      'startDate',
+      'dueDate'
     ],
     ignoreKeys: ['number', 'estimation', 'attachedTo'],
     getVisibleFilters: tracker.function.GetVisibleFilters
@@ -453,7 +484,8 @@ export function createModel (builder: Builder): void {
     TRelatedIssueTarget,
     TTypeEstimation,
     TTypeRemainingTime,
-    TProjectTargetPreference
+    TProjectTargetPreference,
+    TDependencyShiftedNotification
   )
 
   builder.mixin(tracker.class.Project, core.class.Class, activity.mixin.ActivityDoc, {})
@@ -596,6 +628,35 @@ export function createModel (builder: Builder): void {
     tracker.ids.IssueRemovedActivityViewlet
   )
 
+  //  — Activity-Log Remove-Detail Fix.
+  // Three symmetric viewlets so IssueRelation add/remove/update show up in
+  // the issue activity feed with a kind+lag+target.title snapshot instead
+  // of the previous empty "removed related to:" row. The
+  // RelationActivityPresenter reuses IssueRelationPresenter, which is
+  // already registered as the ObjectPresenter for tracker.class.IssueRelation
+  // (models/tracker/src/presenters.ts).
+  builder.createDoc(activity.class.DocUpdateMessageViewlet, core.space.Model, {
+    objectClass: tracker.class.IssueRelation,
+    action: 'create',
+    icon: tracker.icon.Issue,
+    label: tracker.string.AddedRelation,
+    component: tracker.component.RelationActivityPresenter
+  })
+  builder.createDoc(activity.class.DocUpdateMessageViewlet, core.space.Model, {
+    objectClass: tracker.class.IssueRelation,
+    action: 'remove',
+    icon: tracker.icon.Issue,
+    label: tracker.string.RemovedRelation,
+    component: tracker.component.RelationActivityPresenter
+  })
+  builder.createDoc(activity.class.DocUpdateMessageViewlet, core.space.Model, {
+    objectClass: tracker.class.IssueRelation,
+    action: 'update',
+    icon: tracker.icon.Issue,
+    label: tracker.string.UpdatedRelation,
+    component: tracker.component.RelationActivityPresenter
+  })
+
   builder.createDoc(
     activity.class.DocUpdateMessageViewlet,
     core.space.Model,

models/tracker/src/migration.ts

@@ -38,16 +38,20 @@ import { DOMAIN_SPACE } from '@hcengineering/model-core'
 import { DOMAIN_TASK, migrateDefaultStatusesBase } from '@hcengineering/model-task'
 import tags from '@hcengineering/tags'
 import task from '@hcengineering/task'
-import tracker, {
+import {
   type Issue,
+  type IssueRelation,
   type IssueStatus,
   type Project,
   TimeReportDayType,
   trackerId
 } from '@hcengineering/tracker'
+import view, { type ViewOptionModel } from '@hcengineering/view'
 
 import { classicIssueTaskStatuses } from '.'
+import tracker from './plugin'
 import { DOMAIN_TRACKER } from './types'
+import { ganttViewOptions } from './viewlets'
 
 async function createDefaultProject (tx: TxOperations): Promise<void> {
   const current = await tx.findOne(tracker.class.Project, {
@@ -181,6 +185,40 @@ export async function migrateAddStartDate (client: MigrationClient): Promise<voi
   )
 }
 
+// Phase 1 (Visual Polish) adds four new ViewOptions to the IssueGantt
+// viewlet (ganttBarLabelLeft/Inside/Right + ganttQuickInfoOnClick).
+// builder.createDoc is idempotent, so existing workspaces don't pick
+// up the new entries on upgrade-workspace. Re-add the missing ones by
+// merging on `key`, preserving the user's groupBy/orderBy and any
+// already-stored entries. Idempotent — a re-run is a no-op.
+//
+// Mirrors models/card/src/migration.ts:addShowAllVersionsViewOption.
+async function addGanttPhase1ViewOptions (client: MigrationUpgradeClient): Promise<void> {
+  const txOp = new TxOperations(client, core.account.System)
+
+  const viewlets = await client.findAll(view.class.Viewlet, {
+    _id: tracker.viewlet.IssueGantt
+  })
+  if (viewlets.length === 0) return
+
+  const desiredOther = ganttViewOptions().other ?? []
+
+  for (const v of viewlets) {
+    const current = v.viewOptions ?? { groupBy: [], orderBy: [], other: [] }
+    const currentOther: ViewOptionModel[] = current.other ?? []
+    const existingKeys = new Set(currentOther.map((o) => o.key))
+    const missing = desiredOther.filter((o) => !existingKeys.has(o.key))
+    if (missing.length === 0) continue
+
+    await txOp.update(v, {
+      viewOptions: {
+        ...current,
+        other: [...currentOther, ...missing]
+      }
+    })
+  }
+}
+
 async function migrateDefaultStatuses (client: MigrationClient, logger: ModelLogger): Promise<void> {
   const defaultTypeId = tracker.ids.ClassingProjectType
   const typeDescriptor = tracker.descriptors.ProjectType
@@ -375,6 +413,66 @@ async function migrateIssueStatuses (client: MigrationClient): Promise<void> {
   )
 }
 
+/**
+ *  — Activity-Log Remove-Detail Fix.
+ *
+ * Legacy IssueRelation removals went through `ops.removeDoc`, which emits
+ * a bare TxRemoveDoc without parent-issue attachment. The activity
+ * pipeline therefore wrote a DocUpdateMessage whose attachedTo is the
+ * relation itself, never showing up in the issue's activity feed. This
+ * migration re-attaches such DUMs to their parent issue by looking up the
+ * original TxCreateDoc of the now-removed IssueRelation. Best-effort:
+ * when the create-tx is missing (db compaction) the DUM still gets
+ * `updateCollection='relations'` so the activity feed at least shows
+ * "removed dependency" without the target detail.
+ *
+ * Idempotent — once a DUM has Issue-side attachment + `updateCollection`,
+ * the predicate skips it on subsequent runs. The predicate is a local
+ * 4-LOC copy of `isBrokenRelationDum` in
+ * `plugins/tracker-resources/src/components/gantt/lib/relation-activity-migration.ts`
+ * (cannot import across package layers — models/tracker is below
+ * tracker-resources). The helper module is the one with unit tests.
+ */
+async function migrateRelationActivityAttachment (client: MigrationClient): Promise<void> {
+  const issueClass = tracker.class.Issue
+  const dums = await client.find<DocUpdateMessage>(DOMAIN_ACTIVITY, {
+    _class: activity.class.DocUpdateMessage,
+    objectClass: tracker.class.IssueRelation,
+    action: 'remove'
+  })
+  if (dums.length === 0) return
+  for (const dum of dums) {
+    const isBroken = dum.attachedToClass !== issueClass || dum.updateCollection !== 'relations'
+    if (!isBroken) continue
+    // Find the create-tx for this relation. The relation objectId remains
+    // the same across the doc's whole life — that's the key we look up.
+    const createTxes = await client.find<TxCreateDoc<IssueRelation>>(
+      DOMAIN_MODEL_TX,
+      {
+        _class: core.class.TxCreateDoc,
+        objectId: dum.objectId as Ref<IssueRelation>
+      },
+      { limit: 1 }
+    )
+    const createTx = createTxes[0]
+    const patch: Partial<DocUpdateMessage> = {}
+    if (createTx?.attachedTo !== undefined && createTx.attachedToClass !== undefined) {
+      patch.attachedTo = createTx.attachedTo
+      patch.attachedToClass = createTx.attachedToClass
+      patch.updateCollection = createTx.collection ?? 'relations'
+    } else {
+      // Placeholder: ensure the DUM at least shows up in the parent
+      // issue's feed by giving it the collection name; we leave
+      // attachedTo as-is (the runtime DocUpdateMessageObjectValue will
+      // still try buildRemovedDoc with objectId, which often succeeds
+      // even when the create-tx for the *DUM's* attachedTo is gone).
+      if (dum.updateCollection === 'relations') continue
+      patch.updateCollection = 'relations'
+    }
+    await client.update<DocUpdateMessage>(DOMAIN_ACTIVITY, { _id: dum._id }, patch)
+  }
+}
+
 export const trackerOperation: MigrateOperation = {
   async preMigrate (client: MigrationClient, logger: ModelLogger, mode): Promise<void> {
     await tryMigrate(mode, client, trackerId, [
@@ -414,6 +512,22 @@ export const trackerOperation: MigrateOperation = {
         state: 'gantt-add-startdate',
         mode: 'upgrade',
         func: migrateAddStartDate
+      },
+      {
+        // Phase-2 working-days calendar. The property is optional and
+        // additive — every existing Project keeps `workingDaysConfig =
+        // undefined` (legacy calendar-day semantics). The migration entry
+        // exists only so the tracker state-tracker registers the schema
+        // version bump; no data is touched.
+        state: 'gantt-add-working-days-config',
+        mode: 'upgrade',
+        func: async () => {}
+      },
+      {
+        //  — Activity-Log Remove-Detail Fix.
+        state: 'relation-activity-attached-v1',
+        mode: 'upgrade',
+        func: migrateRelationActivityAttachment
       }
     ])
   },
@@ -425,6 +539,10 @@ export const trackerOperation: MigrateOperation = {
           const tx = new TxOperations(client, core.account.System)
           await createDefaults(tx)
         }
+      },
+      {
+        state: 'add-gantt-phase1-view-options',
+        func: addGanttPhase1ViewOptions
       }
     ])
   }

models/tracker/src/plugin.ts

@@ -71,6 +71,7 @@ export default mergeIds(trackerId, tracker, {
     IssueList: '' as Ref<Viewlet>,
     IssueTemplateList: '' as Ref<Viewlet>,
     IssueKanban: '' as Ref<Viewlet>,
+    IssueGantt: '' as Ref<Viewlet>,
     MilestoneList: '' as Ref<Viewlet>,
     ComponentList: '' as Ref<Viewlet>,
     ProjectList: '' as Ref<Viewlet>,
@@ -79,6 +80,8 @@ export default mergeIds(trackerId, tracker, {
   ids: {
     TrackerNotificationGroup: '' as Ref<NotificationGroup>,
     AssigneeNotification: '' as Ref<NotificationType>,
+    //  — Notification on Dependency-Shift.
+    DependencyShiftedNotification: '' as Ref<NotificationType>,
     BaseProjectType: '' as Ref<ProjectType>,
     GuestIssueClassPermission: '' as Ref<Doc>,
     ModulePermissionGroup: '' as Ref<Doc>,

models/tracker/src/presenters.ts

@@ -34,6 +34,22 @@ export function definePresenters (builder: Builder): void {
     presenter: tracker.component.NotificationIssuePresenter
   })
 
+  //
+  // IssueRelation — single-line "FS +Nd → OSTRO-29 Title" presenter so
+  // the activity feed shows which dependency was added or removed
+  // (instead of the generic "Dependency" class label).
+  //
+  builder.mixin(tracker.class.IssueRelation, core.class.Class, view.mixin.ObjectPresenter, {
+    presenter: tracker.component.IssueRelationPresenter
+  })
+
+  //
+  //  — Dependency-Shift bundle notification renderer.
+  //
+  builder.mixin(tracker.class.DependencyShiftedNotification, core.class.Class, view.mixin.ObjectPresenter, {
+    presenter: tracker.component.DependencyShiftedPresenter
+  })
+
   builder.mixin(tracker.class.Issue, core.class.Class, view.mixin.PreviewPresenter, {
     presenter: tracker.component.IssuePreview
   })