Remove page viewers feature from metrics

.github/workflows/ci_tests.yml

@@ -48,22 +48,10 @@ jobs:
           node-version-file: .nvmrc
           cache: npm
 
-      - name: Prepare database
-        run: bundle exec rails db:schema:load
-
-      - name: Install node dependencies
-        run: npm ci
-
-      - name: Precompile assets
-        run: bundle exec rake assets:precompile
-
-      - name: Run unit tests
-        run: bundle exec rails test
-
       - name: Set up Chrome
         uses: browser-actions/setup-chrome@latest
         with:
           chrome-version: stable
 
-      - name: Run system tests
-        run: HEADLESS=1 bundle exec rails test:system
+      - name: Run CI
+        run: bin/ci

.prettierignore

@@ -1,6 +1,8 @@
 .rbenv-vars
 *.rb
 *.rake
+.context/
 app/assets/builds/
 public/vite*/
 tmp/
+vendor/

app/javascript/components/App.tsx

@@ -62,18 +62,18 @@ const App = ({ alert, notice }: { alert: string; notice: string }) => {
           </Routes>
         </ScrollToTop>
         <Footer />
-        <Toaster
-          containerClassName="mt-12"
-          toastOptions={{
-            style: {
-              borderRadius: '10px',
-              background: '#0B1119',
-              color: '#fff',
-              border: '1px solid #1F2A37',
-            },
-          }}
-        />
       </div>
+      <Toaster
+        containerClassName="mt-12"
+        toastOptions={{
+          style: {
+            borderRadius: '10px',
+            background: '#0B1119',
+            color: '#fff',
+            border: '1px solid #1F2A37',
+          },
+        }}
+      />
     </ThemeProvider>
   )
 }

app/javascript/components/Breadcrumb.tsx

@@ -2,12 +2,10 @@ import { Avatar, Breadcrumb as BaseBreadcrumb, Button, Dropdown, Tooltip } from
 import { useNavigate, useSearchParams } from 'react-router-dom'
 import { useMetricWatchersCreate, useMetricWatchersDestroy } from '../queries/user/metricWatchers'
 
-import _ from 'lodash'
 import useCurrentUser from '../hooks/useCurrentUser'
 import { Metric } from '../queries/user/metrics'
 import { Project } from '../queries/user/projects'
 import { useUsersIndex } from '../queries/user/users'
-import { useViewsIndex } from '../queries/user/views'
 import MetricActionsMenu from './MetricActionsMenu'
 import ProjectActionsMenu from './ProjectActionsMenu'
 
@@ -25,10 +23,6 @@ const Breadcrumb = ({ projects, metrics }: { projects: Project[]; metrics: Metri
   const metricId = parseInt(searchParams.get('metric_id') || '')
   const currentMetric = metricId ? metrics.find((metric) => metric.id === metricId) : null
 
-  const { data: views } = useViewsIndex({ metricId })
-  const viewerIds = views ? _.uniq(views.map((view) => view.user_id)) : []
-  const { data: viewers } = useUsersIndex({ ids: viewerIds, enabled: !!currentMetric })
-
   const { data: watchers } = useUsersIndex({ ids: currentMetric?.watcher_ids, enabled: !!currentMetric })
 
   if (!user) return null
@@ -95,24 +89,6 @@ const Breadcrumb = ({ projects, metrics }: { projects: Project[]; metrics: Metri
       {currentProject && (
         <div className="ml-auto flex items-center gap-3">
           {!currentMetric && <ProjectActionsMenu projectId={currentProject.id} />}
-          {viewers && views && views.length > 0 && (
-            <>
-              <p>Seen by:</p>
-              <Avatar.Group>
-                {viewers.map((viewer) => (
-                  <Tooltip key={viewer.id} content={viewer.name} arrow={false}>
-                    <Avatar
-                      img={viewer.image}
-                      rounded
-                      stacked
-                      className="cursor-pointer"
-                      onClick={() => navigate(`/user/users/${viewer.id}`)}
-                    />
-                  </Tooltip>
-                ))}
-              </Avatar.Group>
-            </>
-          )}
           {currentMetric && <MetricActionsMenu metricId={currentMetric.id} projectId={currentProject.id} />}
         </div>
       )}

app/javascript/components/MetricCard.tsx

@@ -1,20 +1,10 @@
 import { Card } from 'flowbite-react'
-import { useEffect } from 'react'
 import { ChartKind } from '../queries/user/charts'
-import { useViewsCreate } from '../queries/user/views'
 import MetricChart from './MetricChart'
 
-const MetricCard = ({ metricId, owners }: { metricId: number; owners?: string[] }) => {
-  const { mutate: trackView } = useViewsCreate()
-
-  useEffect(() => {
-    trackView(metricId)
-  }, [metricId, trackView])
-
-  return (
-    <Card className="mb-3">
-      <MetricChart kind={ChartKind.Area} metricIds={[metricId]} owners={owners} />
-    </Card>
-  )
-}
+const MetricCard = ({ metricId, owners }: { metricId: number; owners?: string[] }) => (
+  <Card className="mb-3">
+    <MetricChart kind={ChartKind.Area} metricIds={[metricId]} owners={owners} />
+  </Card>
+)
 export default MetricCard

app/models/metric.rb

@@ -7,7 +7,6 @@ class Metric < ApplicationRecord
   has_many :reports, dependent: :destroy
   has_many :contributions, dependent: :destroy
   has_many :chart_metrics, dependent: :destroy
-  has_many :views, as: :viewable, dependent: :destroy
 
   validates :name, presence: true
 

app/models/user.rb

@@ -12,7 +12,6 @@ class User < ApplicationRecord
   has_many :metrics, through: :projects
   has_many :notifications, dependent: :destroy
   has_many :owned_organizations, class_name: Organization.to_s, dependent: :restrict_with_error
-  has_many :views, dependent: :destroy
 
   before_save :ensure_api_key
 

bin/ci

@@ -30,9 +30,14 @@ else
   step "Setup: Dependencies" bundle install
   step "Setup: Dependencies" npm install
 fi
-# Kill stale connections to cherry_test so db:test:prepare can purge it
-psql -d cherry_development -c "SELECT pg_terminate_backend(pid) FROM pg_stat_activity WHERE datname = 'cherry_test' AND pid <> pg_backend_pid()" > /dev/null 2>&1 || true
-step "Setup: Database" bin/rails db:test:prepare
+if [ "${CI:-}" = "true" ]; then
+  # Fresh CI database — no need to purge, just create and load schema.
+  step "Setup: Database" bin/rails db:create db:schema:load
+else
+  # Kill stale connections to cherry_test so db:test:prepare can purge it.
+  psql -d cherry_development -c "SELECT pg_terminate_backend(pid) FROM pg_stat_activity WHERE datname = 'cherry_test' AND pid <> pg_backend_pid()" > /dev/null 2>&1 || true
+  step "Setup: Database" bin/rails db:test:prepare
+fi
 
 # Linting
 [ $FAILED -eq 0 ] && step "Style: Ruby" bundle exec rubocop

config/routes.rb

@@ -42,7 +42,6 @@
       resource :settings, only: %i[update]
       resources :subscriptions, only: %i[create]
       resources :users, only: %i[index show]
-      resources :views, only: %i[index create]
     end
 
     constraints(->(request) { request.format == :html }) do

db/migrate/20260417172507_drop_views.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+class DropViews < ActiveRecord::Migration[7.1]
+  def change
+    drop_table :views do |t|
+      t.references :user, null: false, foreign_key: true
+      t.references :viewable, polymorphic: true, null: false
+      t.timestamps
+    end
+  end
+end