comparing push-xpolynpvotzt and master on kot.pink/core

+39 -2

appview/db/db.go

···

       569
       569
        
       		-- indexes for better performance

     

       570
       570
        
       		create index if not exists idx_notifications_recipient_created on notifications(recipient_did, created desc);

     

       571
       571
        
       		create index if not exists idx_notifications_recipient_read on notifications(recipient_did, read);

     

       572
       572
       -
       		create index if not exists idx_stars_created on stars(created);

     

       573
       573
       -
       		create index if not exists idx_stars_repo_at_created on stars(repo_at, created);

     

       574
       572
        
       	`)

     

       575
       573
        
       	if err != nil {

     

       576
       574
        
       		return nil, err

     
···

       1124
       1122
        
       	runMigration(conn, logger, "add-usermentioned-preference", func(tx *sql.Tx) error {

     

       1125
       1123
        
       		_, err := tx.Exec(`

     

       1126
       1124
        
       			alter table notification_preferences add column user_mentioned integer not null default 1;

     

       1125
       1125
       +
       		`)

     

       1126
       1126
       +
       		return err

     

       1127
       1127
       +
       	})

     

       1128
       1128
       +
       

     

       1129
       1129
       +
       	// remove the foreign key constraints from stars.

     

       1130
       1130
       +
       	runMigration(conn, logger, "generalize-stars-subject", func(tx *sql.Tx) error {

     

       1131
       1131
       +
       		_, err := tx.Exec(`

     

       1132
       1132
       +
       			create table stars_new (

     

       1133
       1133
       +
       				id integer primary key autoincrement,

     

       1134
       1134
       +
       				did text not null,

     

       1135
       1135
       +
       				rkey text not null,

     

       1136
       1136
       +
       

     

       1137
       1137
       +
       				subject_at text not null,

     

       1138
       1138
       +
       

     

       1139
       1139
       +
       				created text not null default (strftime('%Y-%m-%dT%H:%M:%SZ', 'now')),

     

       1140
       1140
       +
       				unique(did, rkey),

     

       1141
       1141
       +
       				unique(did, subject_at)

     

       1142
       1142
       +
       			);

     

       1143
       1143
       +
       

     

       1144
       1144
       +
       			insert into stars_new (

     

       1145
       1145
       +
       				id,

     

       1146
       1146
       +
       				did,

     

       1147
       1147
       +
       				rkey,

     

       1148
       1148
       +
       				subject_at,

     

       1149
       1149
       +
       				created

     

       1150
       1150
       +
       			)

     

       1151
       1151
       +
       			select

     

       1152
       1152
       +
       				id,

     

       1153
       1153
       +
       				starred_by_did,

     

       1154
       1154
       +
       				rkey,

     

       1155
       1155
       +
       				repo_at,

     

       1156
       1156
       +
       				created

     

       1157
       1157
       +
       			from stars;

     

       1158
       1158
       +
       

     

       1159
       1159
       +
       			drop table stars;

     

       1160
       1160
       +
       			alter table stars_new rename to stars;

     

       1161
       1161
       +
       

     

       1162
       1162
       +
       			create index if not exists idx_stars_created on stars(created);

     

       1163
       1163
       +
       			create index if not exists idx_stars_subject_at_created on stars(subject_at, created);

     

       1127
       1164
        
       		`)

     

       1128
       1165
        
       		return err

     

       1129
       1166
        
       	})

+3 -3

appview/db/repos.go

···

       208
       208
        
       

     

       209
       209
        
       	starCountQuery := fmt.Sprintf(

     

       210
       210
        
       		`select

     

       211
       211
       -
       			repo_at, count(1)

     

       211
       211
       +
       			subject_at, count(1)

     

       212
       212
        
       		from stars

     

       213
       213
       -
       		where repo_at in (%s)

     

       214
       214
       -
       		group by repo_at`,

     

       213
       213
       +
       		where subject_at in (%s)

     

       214
       214
       +
       		group by subject_at`,

     

       215
       215
        
       		inClause,

     

       216
       216
        
       	)

     

       217
       217
        
       	rows, err = e.Query(starCountQuery, args...)

+39 -99

appview/db/star.go

···

       14
       14
        
       )

     

       15
       15
        
       

     

       16
       16
        
       func AddStar(e Execer, star *models.Star) error {

     

       17
       17
       -
       	query := `insert or ignore into stars (starred_by_did, repo_at, rkey) values (?, ?, ?)`

     

       17
       17
       +
       	query := `insert or ignore into stars (did, subject_at, rkey) values (?, ?, ?)`

     

       18
       18
        
       	_, err := e.Exec(

     

       19
       19
        
       		query,

     

       20
       20
       -
       		star.StarredByDid,

     

       20
       20
       +
       		star.Did,

     

       21
       21
        
       		star.RepoAt.String(),

     

       22
       22
        
       		star.Rkey,

     

       23
       23
        
       	)

     
···

       25
       25
        
       }

     

       26
       26
        
       

     

       27
       27
        
       // Get a star record

     

       28
       28
       -
       func GetStar(e Execer, starredByDid string, repoAt syntax.ATURI) (*models.Star, error) {

     

       28
       28
       +
       func GetStar(e Execer, did string, subjectAt syntax.ATURI) (*models.Star, error) {

     

       29
       29
        
       	query := `

     

       30
       30
       -
       	select starred_by_did, repo_at, created, rkey

     

       30
       30
       +
       	select did, subject_at, created, rkey

     

       31
       31
        
       	from stars

     

       32
       32
       -
       	where starred_by_did = ? and repo_at = ?`

     

       33
       33
       -
       	row := e.QueryRow(query, starredByDid, repoAt)

     

       32
       32
       +
       	where did = ? and subject_at = ?`

     

       33
       33
       +
       	row := e.QueryRow(query, did, subjectAt)

     

       34
       34
        
       

     

       35
       35
        
       	var star models.Star

     

       36
       36
        
       	var created string

     

       37
       37
       -
       	err := row.Scan(&star.StarredByDid, &star.RepoAt, &created, &star.Rkey)

     

       37
       37
       +
       	err := row.Scan(&star.Did, &star.RepoAt, &created, &star.Rkey)

     

       38
       38
        
       	if err != nil {

     

       39
       39
        
       		return nil, err

     

       40
       40
        
       	}

     
···

       51
       51
        
       }

     

       52
       52
        
       

     

       53
       53
        
       // Remove a star

     

       54
       54
       -
       func DeleteStar(e Execer, starredByDid string, repoAt syntax.ATURI) error {

     

       55
       55
       -
       	_, err := e.Exec(`delete from stars where starred_by_did = ? and repo_at = ?`, starredByDid, repoAt)

     

       54
       54
       +
       func DeleteStar(e Execer, did string, subjectAt syntax.ATURI) error {

     

       55
       55
       +
       	_, err := e.Exec(`delete from stars where did = ? and subject_at = ?`, did, subjectAt)

     

       56
       56
        
       	return err

     

       57
       57
        
       }

     

       58
       58
        
       

     

       59
       59
        
       // Remove a star

     

       60
       60
       -
       func DeleteStarByRkey(e Execer, starredByDid string, rkey string) error {

     

       61
       61
       -
       	_, err := e.Exec(`delete from stars where starred_by_did = ? and rkey = ?`, starredByDid, rkey)

     

       60
       60
       +
       func DeleteStarByRkey(e Execer, did string, rkey string) error {

     

       61
       61
       +
       	_, err := e.Exec(`delete from stars where did = ? and rkey = ?`, did, rkey)

     

       62
       62
        
       	return err

     

       63
       63
        
       }

     

       64
       64
        
       

     

       65
       65
       -
       func GetStarCount(e Execer, repoAt syntax.ATURI) (int, error) {

     

       65
       65
       +
       func GetStarCount(e Execer, subjectAt syntax.ATURI) (int, error) {

     

       66
       66
        
       	stars := 0

     

       67
       67
        
       	err := e.QueryRow(

     

       68
       68
       -
       		`select count(starred_by_did) from stars where repo_at = ?`, repoAt).Scan(&stars)

     

       68
       68
       +
       		`select count(did) from stars where subject_at = ?`, subjectAt).Scan(&stars)

     

       69
       69
        
       	if err != nil {

     

       70
       70
        
       		return 0, err

     

       71
       71
        
       	}

     
···

       89
       89
        
       	}

     

       90
       90
        
       

     

       91
       91
        
       	query := fmt.Sprintf(`

     

       92
       92
       -
       		SELECT repo_at

     

       92
       92
       +
       		SELECT subject_at

     

       93
       93
        
       		FROM stars

     

       94
       94
       -
       		WHERE starred_by_did = ? AND repo_at IN (%s)

     

       94
       94
       +
       		WHERE did = ? AND subject_at IN (%s)

     

       95
       95
        
       	`, strings.Join(placeholders, ","))

     

       96
       96
        
       

     

       97
       97
        
       	rows, err := e.Query(query, args...)

     
···

       118
       118
        
       	return result, nil

     

       119
       119
        
       }

     

       120
       120
        
       

     

       121
       121
       -
       func GetStarStatus(e Execer, userDid string, repoAt syntax.ATURI) bool {

     

       122
       122
       -
       	statuses, err := getStarStatuses(e, userDid, []syntax.ATURI{repoAt})

     

       121
       121
       +
       func GetStarStatus(e Execer, userDid string, subjectAt syntax.ATURI) bool {

     

       122
       122
       +
       	statuses, err := getStarStatuses(e, userDid, []syntax.ATURI{subjectAt})

     

       123
       123
        
       	if err != nil {

     

       124
       124
        
       		return false

     

       125
       125
        
       	}

     

       126
       126
       -
       	return statuses[repoAt.String()]

     

       126
       126
       +
       	return statuses[subjectAt.String()]

     

       127
       127
        
       }

     

       128
       128
        
       

     

       129
       129
        
       // GetStarStatuses returns a map of repo URIs to star status for a given user

     

       130
       130
       -
       func GetStarStatuses(e Execer, userDid string, repoAts []syntax.ATURI) (map[string]bool, error) {

     

       131
       131
       -
       	return getStarStatuses(e, userDid, repoAts)

     

       130
       130
       +
       func GetStarStatuses(e Execer, userDid string, subjectAts []syntax.ATURI) (map[string]bool, error) {

     

       131
       131
       +
       	return getStarStatuses(e, userDid, subjectAts)

     

       132
       132
        
       }

     

       133
       133
       -
       func GetStars(e Execer, limit int, filters ...filter) ([]models.Star, error) {

     

       133
       133
       +
       

     

       134
       134
       +
       // GetRepoStars return a list of stars each holding target repository.

     

       135
       135
       +
       // If there isn't known repo with starred at-uri, those stars will be ignored.

     

       136
       136
       +
       func GetRepoStars(e Execer, limit int, filters ...filter) ([]models.RepoStar, error) {

     

       134
       137
        
       	var conditions []string

     

       135
       138
        
       	var args []any

     

       136
       139
        
       	for _, filter := range filters {

     
···

       149
       152
        
       	}

     

       150
       153
        
       

     

       151
       154
        
       	repoQuery := fmt.Sprintf(

     

       152
       152
       -
       		`select starred_by_did, repo_at, created, rkey

     

       155
       155
       +
       		`select did, subject_at, created, rkey

     

       153
       156
        
       		from stars

     

       154
       157
        
       		%s

     

       155
       158
        
       		order by created desc

     
···

       166
       169
        
       	for rows.Next() {

     

       167
       170
        
       		var star models.Star

     

       168
       171
        
       		var created string

     

       169
       169
       -
       		err := rows.Scan(&star.StarredByDid, &star.RepoAt, &created, &star.Rkey)

     

       172
       172
       +
       		err := rows.Scan(&star.Did, &star.RepoAt, &created, &star.Rkey)

     

       170
       173
        
       		if err != nil {

     

       171
       174
        
       			return nil, err

     

       172
       175
        
       		}

     
···

       197
       200
        
       		return nil, err

     

       198
       201
        
       	}

     

       199
       202
        
       

     

       203
       203
       +
       	var repoStars []models.RepoStar

     

       200
       204
        
       	for _, r := range repos {

     

       201
       205
        
       		if stars, ok := starMap[string(r.RepoAt())]; ok {

     

       202
       202
       -
       			for i := range stars {

     

       203
       203
       -
       				stars[i].Repo = &r

     

       206
       206
       +
       			for _, star := range stars {

     

       207
       207
       +
       				repoStars = append(repoStars, models.RepoStar{

     

       208
       208
       +
       					Star: star,

     

       209
       209
       +
       					Repo: &r,

     

       210
       210
       +
       				})

     

       204
       211
        
       			}

     

       205
       212
        
       		}

     

       206
       213
        
       	}

     

       207
       214
        
       

     

       208
       208
       -
       	var stars []models.Star

     

       209
       209
       -
       	for _, s := range starMap {

     

       210
       210
       -
       		stars = append(stars, s...)

     

       211
       211
       -
       	}

     

       212
       212
       -
       

     

       213
       213
       -
       	slices.SortFunc(stars, func(a, b models.Star) int {

     

       215
       215
       +
       	slices.SortFunc(repoStars, func(a, b models.RepoStar) int {

     

       214
       216
        
       		if a.Created.After(b.Created) {

     

       215
       217
        
       			return -1

     

       216
       218
        
       		}

     
···

       220
       222
        
       		return 0

     

       221
       223
        
       	})

     

       222
       224
        
       

     

       223
       223
       -
       	return stars, nil

     

       225
       225
       +
       	return repoStars, nil

     

       224
       226
        
       }

     

       225
       227
        
       

     

       226
       228
        
       func CountStars(e Execer, filters ...filter) (int64, error) {

     
···

       247
       249
        
       	return count, nil

     

       248
       250
        
       }

     

       249
       251
        
       

     

       250
       250
       -
       func GetAllStars(e Execer, limit int) ([]models.Star, error) {

     

       251
       251
       -
       	var stars []models.Star

     

       252
       252
       -
       

     

       253
       253
       -
       	rows, err := e.Query(`

     

       254
       254
       -
       		select

     

       255
       255
       -
       			s.starred_by_did,

     

       256
       256
       -
       			s.repo_at,

     

       257
       257
       -
       			s.rkey,

     

       258
       258
       -
       			s.created,

     

       259
       259
       -
       			r.did,

     

       260
       260
       -
       			r.name,

     

       261
       261
       -
       			r.knot,

     

       262
       262
       -
       			r.rkey,

     

       263
       263
       -
       			r.created

     

       264
       264
       -
       		from stars s

     

       265
       265
       -
       		join repos r on s.repo_at = r.at_uri

     

       266
       266
       -
       	`)

     

       267
       267
       -
       

     

       268
       268
       -
       	if err != nil {

     

       269
       269
       -
       		return nil, err

     

       270
       270
       -
       	}

     

       271
       271
       -
       	defer rows.Close()

     

       272
       272
       -
       

     

       273
       273
       -
       	for rows.Next() {

     

       274
       274
       -
       		var star models.Star

     

       275
       275
       -
       		var repo models.Repo

     

       276
       276
       -
       		var starCreatedAt, repoCreatedAt string

     

       277
       277
       -
       

     

       278
       278
       -
       		if err := rows.Scan(

     

       279
       279
       -
       			&star.StarredByDid,

     

       280
       280
       -
       			&star.RepoAt,

     

       281
       281
       -
       			&star.Rkey,

     

       282
       282
       -
       			&starCreatedAt,

     

       283
       283
       -
       			&repo.Did,

     

       284
       284
       -
       			&repo.Name,

     

       285
       285
       -
       			&repo.Knot,

     

       286
       286
       -
       			&repo.Rkey,

     

       287
       287
       -
       			&repoCreatedAt,

     

       288
       288
       -
       		); err != nil {

     

       289
       289
       -
       			return nil, err

     

       290
       290
       -
       		}

     

       291
       291
       -
       

     

       292
       292
       -
       		star.Created, err = time.Parse(time.RFC3339, starCreatedAt)

     

       293
       293
       -
       		if err != nil {

     

       294
       294
       -
       			star.Created = time.Now()

     

       295
       295
       -
       		}

     

       296
       296
       -
       		repo.Created, err = time.Parse(time.RFC3339, repoCreatedAt)

     

       297
       297
       -
       		if err != nil {

     

       298
       298
       -
       			repo.Created = time.Now()

     

       299
       299
       -
       		}

     

       300
       300
       -
       		star.Repo = &repo

     

       301
       301
       -
       

     

       302
       302
       -
       		stars = append(stars, star)

     

       303
       303
       -
       	}

     

       304
       304
       -
       

     

       305
       305
       -
       	if err := rows.Err(); err != nil {

     

       306
       306
       -
       		return nil, err

     

       307
       307
       -
       	}

     

       308
       308
       -
       

     

       309
       309
       -
       	return stars, nil

     

       310
       310
       -
       }

     

       311
       311
       -
       

     

       312
       252
        
       // GetTopStarredReposLastWeek returns the top 8 most starred repositories from the last week

     

       313
       253
        
       func GetTopStarredReposLastWeek(e Execer) ([]models.Repo, error) {

     

       314
       254
        
       	// first, get the top repo URIs by star count from the last week

     

       315
       255
        
       	query := `

     

       316
       256
        
       		with recent_starred_repos as (

     

       317
       317
       -
       			select distinct repo_at

     

       257
       257
       +
       			select distinct subject_at

     

       318
       258
        
       			from stars

     

       319
       259
        
       			where created >= datetime('now', '-7 days')

     

       320
       260
        
       		),

     

       321
       261
        
       		repo_star_counts as (

     

       322
       262
        
       			select

     

       323
       323
       -
       				s.repo_at,

     

       263
       263
       +
       				s.subject_at,

     

       324
       264
        
       				count(*) as stars_gained_last_week

     

       325
       265
        
       			from stars s

     

       326
       326
       -
       			join recent_starred_repos rsr on s.repo_at = rsr.repo_at

     

       266
       266
       +
       			join recent_starred_repos rsr on s.subject_at = rsr.subject_at

     

       327
       267
        
       			where s.created >= datetime('now', '-7 days')

     

       328
       328
       -
       			group by s.repo_at

     

       268
       268
       +
       			group by s.subject_at

     

       329
       269
        
       		)

     

       330
       330
       -
       		select rsc.repo_at

     

       270
       270
       +
       		select rsc.subject_at

     

       331
       271
        
       		from repo_star_counts rsc

     

       332
       272
        
       		order by rsc.stars_gained_last_week desc

     

       333
       273
        
       		limit 8

+3 -13

appview/db/timeline.go

···

       146
       146
        
       func getTimelineStars(e Execer, limit int, loggedInUserDid string, userIsFollowing []string) ([]models.TimelineEvent, error) {

     

       147
       147
        
       	filters := make([]filter, 0)

     

       148
       148
        
       	if userIsFollowing != nil {

     

       149
       149
       -
       		filters = append(filters, FilterIn("starred_by_did", userIsFollowing))

     

       149
       149
       +
       		filters = append(filters, FilterIn("did", userIsFollowing))

     

       150
       150
        
       	}

     

       151
       151
        
       

     

       152
       152
       -
       	stars, err := GetStars(e, limit, filters...)

     

       152
       152
       +
       	stars, err := GetRepoStars(e, limit, filters...)

     

       153
       153
        
       	if err != nil {

     

       154
       154
        
       		return nil, err

     

       155
       155
        
       	}

     

       156
       156
        
       

     

       157
       157
       -
       	// filter star records without a repo

     

       158
       158
       -
       	n := 0

     

       159
       159
       -
       	for _, s := range stars {

     

       160
       160
       -
       		if s.Repo != nil {

     

       161
       161
       -
       			stars[n] = s

     

       162
       162
       -
       			n++

     

       163
       163
       -
       		}

     

       164
       164
       -
       	}

     

       165
       165
       -
       	stars = stars[:n]

     

       166
       166
       -
       

     

       167
       157
        
       	var repos []models.Repo

     

       168
       158
        
       	for _, s := range stars {

     

       169
       159
        
       		repos = append(repos, *s.Repo)

     
···

       179
       169
        
       		isStarred, starCount := getRepoStarInfo(s.Repo, starStatuses)

     

       180
       170
        
       

     

       181
       171
        
       		events = append(events, models.TimelineEvent{

     

       182
       182
       -
       			Star:      &s,

     

       172
       172
       +
       			RepoStar:  &s,

     

       183
       173
        
       			EventAt:   s.Created,

     

       184
       174
        
       			IsStarred: isStarred,

     

       185
       175
        
       			StarCount: starCount,

+3 -3

appview/ingester.go

···

       121
       121
        
       			return err

     

       122
       122
        
       		}

     

       123
       123
        
       		err = db.AddStar(i.Db, &models.Star{

     

       124
       124
       -
       			StarredByDid: did,

     

       125
       125
       -
       			RepoAt:       subjectUri,

     

       126
       126
       -
       			Rkey:         e.Commit.RKey,

     

       124
       124
       +
       			Did:    did,

     

       125
       125
       +
       			RepoAt: subjectUri,

     

       126
       126
       +
       			Rkey:   e.Commit.RKey,

     

       127
       127
        
       		})

     

       128
       128
        
       	case jmodels.CommitOperationDelete:

     

       129
       129
        
       		err = db.DeleteStarByRkey(i.Db, did, e.Commit.RKey)

+14 -5

appview/models/star.go

···

       7
       7
        
       )

     

       8
       8
        
       

     

       9
       9
        
       type Star struct {

     

       10
       10
       -
       	StarredByDid string

     

       11
       11
       -
       	RepoAt       syntax.ATURI

     

       12
       12
       -
       	Created      time.Time

     

       13
       13
       -
       	Rkey         string

     

       10
       10
       +
       	Did     string

     

       11
       11
       +
       	RepoAt  syntax.ATURI

     

       12
       12
       +
       	Created time.Time

     

       13
       13
       +
       	Rkey    string

     

       14
       14
       +
       }

     

       14
       15
        
       

     

       15
       15
       -
       	// optionally, populate this when querying for reverse mappings

     

       16
       16
       +
       // RepoStar is used for reverse mapping to repos

     

       17
       17
       +
       type RepoStar struct {

     

       18
       18
       +
       	Star

     

       16
       19
        
       	Repo *Repo

     

       17
       20
        
       }

     

       21
       21
       +
       

     

       22
       22
       +
       // StringStar is used for reverse mapping to strings

     

       23
       23
       +
       type StringStar struct {

     

       24
       24
       +
       	Star

     

       25
       25
       +
       	String *String

     

       26
       26
       +
       }

+1 -1

appview/models/string.go

···

       22
       22
        
       	Edited      *time.Time

     

       23
       23
        
       }

     

       24
       24
        
       

     

       25
       25
       -
       func (s *String) StringAt() syntax.ATURI {

     

       25
       25
       +
       func (s *String) AtUri() syntax.ATURI {

     

       26
       26
        
       	return syntax.ATURI(fmt.Sprintf("at://%s/%s/%s", s.Did, tangled.StringNSID, s.Rkey))

     

       27
       27
        
       }

     

       28
       28

+1 -1

appview/models/timeline.go

···

       5
       5
        
       type TimelineEvent struct {

     

       6
       6
        
       	*Repo

     

       7
       7
        
       	*Follow

     

       8
       8
       -
       	*Star

     

       8
       8
       +
       	*RepoStar

     

       9
       9
        
       

     

       10
       10
        
       	EventAt time.Time

     

       11
       11

+6 -1

appview/notify/db/db.go

···

       7
       7
        
       	"slices"

     

       8
       8
        
       

     

       9
       9
        
       	"github.com/bluesky-social/indigo/atproto/syntax"

     

       10
       10
       +
       	"tangled.org/core/api/tangled"

     

       10
       11
        
       	"tangled.org/core/appview/db"

     

       11
       12
        
       	"tangled.org/core/appview/models"

     

       12
       13
        
       	"tangled.org/core/appview/notify"

     
···

       36
       37
        
       }

     

       37
       38
        
       

     

       38
       39
        
       func (n *databaseNotifier) NewStar(ctx context.Context, star *models.Star) {

     

       40
       40
       +
       	if star.RepoAt.Collection().String() != tangled.RepoNSID {

     

       41
       41
       +
       		// skip string stars for now

     

       42
       42
       +
       		return

     

       43
       43
       +
       	}

     

       39
       44
        
       	var err error

     

       40
       45
        
       	repo, err := db.GetRepo(n.db, db.FilterEq("at_uri", string(star.RepoAt)))

     

       41
       46
        
       	if err != nil {

     
···

       43
       48
        
       		return

     

       44
       49
        
       	}

     

       45
       50
        
       

     

       46
       46
       -
       	actorDid := syntax.DID(star.StarredByDid)

     

       51
       51
       +
       	actorDid := syntax.DID(star.Did)

     

       47
       52
        
       	recipients := []syntax.DID{syntax.DID(repo.Did)}

     

       48
       53
        
       	eventType := models.NotificationTypeRepoStarred

     

       49
       54
        
       	entityType := "repo"

+2 -2

appview/notify/posthog/notifier.go

···

       37
       37
        
       

     

       38
       38
        
       func (n *posthogNotifier) NewStar(ctx context.Context, star *models.Star) {

     

       39
       39
        
       	err := n.client.Enqueue(posthog.Capture{

     

       40
       40
       -
       		DistinctId: star.StarredByDid,

     

       40
       40
       +
       		DistinctId: star.Did,

     

       41
       41
        
       		Event:      "star",

     

       42
       42
        
       		Properties: posthog.Properties{"repo_at": star.RepoAt.String()},

     

       43
       43
        
       	})

     
···

       48
       48
        
       

     

       49
       49
        
       func (n *posthogNotifier) DeleteStar(ctx context.Context, star *models.Star) {

     

       50
       50
        
       	err := n.client.Enqueue(posthog.Capture{

     

       51
       51
       -
       		DistinctId: star.StarredByDid,

     

       51
       51
       +
       		DistinctId: star.Did,

     

       52
       52
        
       		Event:      "unstar",

     

       53
       53
        
       		Properties: posthog.Properties{"repo_at": star.RepoAt.String()},

     

       54
       54
        
       	})

+1 -1

appview/pages/markup/extension/atlink.go

···

       89
       89
        
       	if entering {

     

       90
       90
        
       		w.WriteString(`<a href="/@`)

     

       91
       91
        
       		w.WriteString(n.(*AtNode).Handle)

     

       92
       92
       -
       		w.WriteString(`" class="mention">`)

     

       92
       92
       +
       		w.WriteString(`" class="mention font-bold">`)

     

       93
       93
        
       	} else {

     

       94
       94
        
       		w.WriteString("</a>")

     

       95
       95
        
       	}

+8 -6

appview/pages/pages.go

···

       625
       625
        
       	return p.executePlain("user/fragments/editPins", w, params)

     

       626
       626
        
       }

     

       627
       627
        
       

     

       628
       628
       -
       type RepoStarFragmentParams struct {

     

       628
       628
       +
       type StarBtnFragmentParams struct {

     

       629
       629
        
       	IsStarred bool

     

       630
       630
       -
       	RepoAt    syntax.ATURI

     

       631
       631
       -
       	Stats     models.RepoStats

     

       630
       630
       +
       	SubjectAt syntax.ATURI

     

       631
       631
       +
       	StarCount int

     

       632
       632
        
       }

     

       633
       633
        
       

     

       634
       634
       -
       func (p *Pages) RepoStarFragment(w io.Writer, params RepoStarFragmentParams) error {

     

       635
       635
       -
       	return p.executePlain("repo/fragments/repoStar", w, params)

     

       634
       634
       +
       func (p *Pages) StarBtnFragment(w io.Writer, params StarBtnFragmentParams) error {

     

       635
       635
       +
       	return p.executePlain("fragments/starBtn", w, params)

     

       636
       636
        
       }

     

       637
       637
        
       

     

       638
       638
        
       type RepoIndexParams struct {

     
···

       1376
       1376
        
       	ShowRendered     bool

     

       1377
       1377
        
       	RenderToggle     bool

     

       1378
       1378
        
       	RenderedContents template.HTML

     

       1379
       1379
       -
       	String           models.String

     

       1379
       1379
       +
       	String           *models.String

     

       1380
       1380
        
       	Stats            models.StringStats

     

       1381
       1381
       +
       	IsStarred        bool

     

       1382
       1382
       +
       	StarCount        int

     

       1381
       1383
        
       	Owner            identity.Identity

     

       1382
       1384
        
       }

     

       1383
       1385

+28

appview/pages/templates/fragments/starBtn.html

···

       1
       1
       +
       {{ define "fragments/starBtn" }}

     

       2
       2
       +
           <button

     

       3
       3
       +
               id="starBtn"

     

       4
       4
       +
               class="btn disabled:opacity-50 disabled:cursor-not-allowed flex gap-2 items-center group"

     

       5
       5
       +
               data-star-subject-at="{{ .SubjectAt }}"

     

       6
       6
       +
               {{ if .IsStarred }}

     

       7
       7
       +
                   hx-delete="/star?subject={{ .SubjectAt }}&countHint={{ .StarCount }}"

     

       8
       8
       +
               {{ else }}

     

       9
       9
       +
                   hx-post="/star?subject={{ .SubjectAt }}&countHint={{ .StarCount }}"

     

       10
       10
       +
               {{ end }}

     

       11
       11
       +
       

     

       12
       12
       +
               hx-trigger="click"

     

       13
       13
       +
               hx-target="this"

     

       14
       14
       +
               hx-swap="outerHTML"

     

       15
       15
       +
               hx-swap-oob='outerHTML:#starBtn[data-star-subject-at="{{ .SubjectAt }}"]'

     

       16
       16
       +
               hx-disabled-elt="#starBtn"

     

       17
       17
       +
           >

     

       18
       18
       +
               {{ if .IsStarred }}

     

       19
       19
       +
                   {{ i "star" "w-4 h-4 fill-current" }}

     

       20
       20
       +
               {{ else }}

     

       21
       21
       +
                   {{ i "star" "w-4 h-4" }}

     

       22
       22
       +
               {{ end }}

     

       23
       23
       +
               <span class="text-sm">

     

       24
       24
       +
                   {{ .StarCount }}

     

       25
       25
       +
               </span>

     

       26
       26
       +
               {{ i "loader-circle" "w-4 h-4 animate-spin hidden group-[.htmx-request]:inline" }}

     

       27
       27
       +
           </button>

     

       28
       28
       +
       {{ end }}

+4 -1

appview/pages/templates/layouts/repobase.html

···

       49
       49
        
               </div>

     

       50
       50
        
       

     

       51
       51
        
               <div class="w-full sm:w-fit grid grid-cols-3 gap-2 z-auto">

     

       52
       52
       -
                 {{ template "repo/fragments/repoStar" .RepoInfo }}

     

       52
       52
       +
                 {{ template "fragments/starBtn"

     

       53
       53
       +
                   (dict "SubjectAt" .RepoInfo.RepoAt

     

       54
       54
       +
                         "IsStarred" .RepoInfo.IsStarred

     

       55
       55
       +
                         "StarCount" .RepoInfo.Stats.StarCount) }}

     

       53
       56
        
                 <a

     

       54
       57
        
                   class="btn text-sm no-underline hover:no-underline flex items-center gap-2 group"

     

       55
       58
        
                   hx-boost="true"

+1

appview/pages/templates/repo/fork.html

···

       25
       25
        
                       value="{{ . }}"

     

       26
       26
        
                       class="mr-2"

     

       27
       27
        
                       id="domain-{{ . }}"

     

       28
       28
       +
                       {{if eq (len $.Knots) 1}}checked{{end}}                

     

       28
       29
        
                       />

     

       29
       30
        
                   <label for="domain-{{ . }}" class="dark:text-white">{{ . }}</label>

     

       30
       31
        
                 </div>

-26

appview/pages/templates/repo/fragments/repoStar.html

···

       1
       1
       -
       {{ define "repo/fragments/repoStar" }}

     

       2
       2
       -
           <button

     

       3
       3
       -
               id="starBtn"

     

       4
       4
       -
               class="btn disabled:opacity-50 disabled:cursor-not-allowed flex gap-2 items-center group"

     

       5
       5
       -
               {{ if .IsStarred }}

     

       6
       6
       -
                   hx-delete="/star?subject={{ .RepoAt }}&countHint={{ .Stats.StarCount }}"

     

       7
       7
       -
               {{ else }}

     

       8
       8
       -
                   hx-post="/star?subject={{ .RepoAt }}&countHint={{ .Stats.StarCount }}"

     

       9
       9
       -
               {{ end }}

     

       10
       10
       -
       

     

       11
       11
       -
               hx-trigger="click"

     

       12
       12
       -
               hx-target="this"

     

       13
       13
       -
               hx-swap="outerHTML"

     

       14
       14
       -
               hx-disabled-elt="#starBtn"

     

       15
       15
       -
           >

     

       16
       16
       -
               {{ if .IsStarred }}

     

       17
       17
       -
                   {{ i "star" "w-4 h-4 fill-current" }}

     

       18
       18
       -
               {{ else }}

     

       19
       19
       -
                   {{ i "star" "w-4 h-4" }}

     

       20
       20
       -
               {{ end }}

     

       21
       21
       -
               <span class="text-sm">

     

       22
       22
       -
                   {{ .Stats.StarCount }}

     

       23
       23
       -
               </span>

     

       24
       24
       -
               {{ i "loader-circle" "w-4 h-4 animate-spin hidden group-[.htmx-request]:inline" }}

     

       25
       25
       -
           </button>

     

       26
       26
       -
       {{ end }}

+1

appview/pages/templates/repo/new.html

···

       155
       155
        
                   class="mr-2"

     

       156
       156
        
                   id="domain-{{ . }}"

     

       157
       157
        
                   required

     

       158
       158
       +
                   {{if eq (len $.Knots) 1}}checked{{end}}

     

       158
       159
        
                 />

     

       159
       160
        
                 <label for="domain-{{ . }}" class="dark:text-white lowercase">{{ . }}</label>

     

       160
       161
        
               </div>

+8 -4

appview/pages/templates/strings/string.html

···

       17
       17
        
               <span class="select-none">/</span>

     

       18
       18
        
               <a href="/strings/{{ $ownerId }}/{{ .String.Rkey }}" class="font-bold">{{ .String.Filename }}</a>

     

       19
       19
        
             </div>

     

       20
       20
       -
             {{ if and .LoggedInUser (eq .LoggedInUser.Did .String.Did) }}

     

       21
       21
       -
               <div class="flex gap-2 text-base">

     

       20
       20
       +
             <div class="flex gap-2 text-base">

     

       21
       21
       +
               {{ if and .LoggedInUser (eq .LoggedInUser.Did .String.Did) }}

     

       22
       22
        
                 <a class="btn flex items-center gap-2 no-underline hover:no-underline p-2 group"

     

       23
       23
        
                    hx-boost="true"

     

       24
       24
        
                    href="/strings/{{ .String.Did }}/{{ .String.Rkey }}/edit">

     
···

       37
       37
        
                   <span class="hidden md:inline">delete</span>

     

       38
       38
        
                   {{ i "loader-circle" "w-4 h-4 animate-spin hidden group-[.htmx-request]:inline" }}

     

       39
       39
        
                 </button>

     

       40
       40
       -
               </div>

     

       41
       41
       -
             {{ end }}

     

       40
       40
       +
               {{ end }}

     

       41
       41
       +
               {{ template "fragments/starBtn"

     

       42
       42
       +
                 (dict "SubjectAt" .String.AtUri

     

       43
       43
       +
                       "IsStarred" .IsStarred

     

       44
       44
       +
                       "StarCount" .StarCount) }}

     

       45
       45
       +
             </div>

     

       42
       46
        
           </div>

     

       43
       47
        
           <span>

     

       44
       48
        
             {{ with .String.Description }}

+1 -2

appview/pages/templates/timeline/fragments/goodfirstissues.html

···

       3
       3
        
           <a href="/goodfirstissues" class="no-underline hover:no-underline">

     

       4
       4
        
             <div class="flex items-center justify-between gap-2 bg-purple-200 dark:bg-purple-900 border border-purple-400 dark:border-purple-500 rounded mb-4 py-4 px-6 ">

     

       5
       5
        
               <div class="flex-1 flex flex-col gap-2">

     

       6
       6
       -
                 <div class="text-purple-500 dark:text-purple-400">Oct 2025</div>

     

       7
       6
        
                 <p>

     

       8
       8
       -
                   Make your first contribution to an open-source project this October.

     

       7
       7
       +
                   Make your first contribution to an open-source project.

     

       9
       8
        
                   <em>good-first-issue</em> helps new contributors find easy ways to

     

       10
       9
        
                   start contributing to open-source projects.

     

       11
       10
        
                 </p>

+4 -4

appview/pages/templates/timeline/fragments/timeline.html

···

       52
       52
        
             <span class="text-gray-700 dark:text-gray-400 text-xs">{{ template "repo/fragments/time" $repo.Created }}</span>

     

       53
       53
        
           </div>

     

       54
       54
        
         {{ with $repo }}

     

       55
       55
       -
           {{ template "user/fragments/repoCard" (list $root . true true (dict "IsStarred" $event.IsStarred "RepoAt" .RepoAt "Stats" (dict "StarCount" $event.StarCount))) }}

     

       55
       55
       +
           {{ template "user/fragments/repoCard" (list $root . true true (dict "IsStarred" $event.IsStarred "SubjectAt" .RepoAt "StarCount" $event.StarCount)) }}

     

       56
       56
        
         {{ end }}

     

       57
       57
        
       {{ end }}

     

       58
       58
        
       

     

       59
       59
        
       {{ define "timeline/fragments/starEvent" }}

     

       60
       60
        
         {{ $root := index . 0 }}

     

       61
       61
        
         {{ $event := index . 1 }}

     

       62
       62
       -
         {{ $star := $event.Star }}

     

       62
       62
       +
         {{ $star := $event.RepoStar }}

     

       63
       63
        
         {{ with $star }}

     

       64
       64
       -
           {{ $starrerHandle :=  resolve .StarredByDid }}

     

       64
       64
       +
           {{ $starrerHandle := resolve .Did }}

     

       65
       65
        
           {{ $repoOwnerHandle :=  resolve .Repo.Did }}

     

       66
       66
        
           <div class="pl-6 py-2 bg-white dark:bg-gray-800 text-gray-600 dark:text-gray-300 flex flex-wrap items-center gap-2 text-sm">

     

       67
       67
        
               {{ template "user/fragments/picHandleLink" $starrerHandle }}

     
···

       72
       72
        
               <span class="text-gray-700 dark:text-gray-400 text-xs">{{ template "repo/fragments/time" .Created }}</span>

     

       73
       73
        
           </div>

     

       74
       74
        
           {{ with .Repo }}

     

       75
       75
       -
             {{ template "user/fragments/repoCard" (list $root . true true (dict "IsStarred" $event.IsStarred "RepoAt" .RepoAt "Stats" (dict "StarCount" $event.StarCount))) }}

     

       75
       75
       +
             {{ template "user/fragments/repoCard" (list $root . true true (dict "IsStarred" $event.IsStarred "SubjectAt" .RepoAt "StarCount" $event.StarCount)) }}

     

       76
       76
        
           {{ end }}

     

       77
       77
        
         {{ end }}

     

       78
       78
        
       {{ end }}

+2 -1

appview/pages/templates/user/fragments/repoCard.html

···

       1
       1
        
       {{ define "user/fragments/repoCard" }}

     

       2
       2
       +
         {{/* root, repo, fullName [,starButton [,starData]] */}}

     

       2
       3
        
         {{ $root := index . 0 }}

     

       3
       4
        
         {{ $repo := index . 1 }}

     

       4
       5
        
         {{ $fullName := index . 2 }}

     
···

       29
       30
        
               </div>

     

       30
       31
        
               {{ if and $starButton $root.LoggedInUser }}

     

       31
       32
        
               <div class="shrink-0">

     

       32
       32
       -
                 {{ template "repo/fragments/repoStar" $starData }}

     

       33
       33
       +
                 {{ template "fragments/starBtn" $starData }}

     

       33
       34
        
               </div>

     

       34
       35
        
               {{ end }}

     

       35
       36
        
             </div>

+3 -5

appview/state/profile.go

···

       66
       66
        
       		return nil, fmt.Errorf("failed to get string count: %w", err)

     

       67
       67
        
       	}

     

       68
       68
        
       

     

       69
       69
       -
       	starredCount, err := db.CountStars(s.db, db.FilterEq("starred_by_did", did))

     

       69
       69
       +
       	starredCount, err := db.CountStars(s.db, db.FilterEq("did", did))

     

       70
       70
        
       	if err != nil {

     

       71
       71
        
       		return nil, fmt.Errorf("failed to get starred repo count: %w", err)

     

       72
       72
        
       	}

     
···

       211
       211
        
       	}

     

       212
       212
        
       	l = l.With("profileDid", profile.UserDid, "profileHandle", profile.UserHandle)

     

       213
       213
        
       

     

       214
       214
       -
       	stars, err := db.GetStars(s.db, 0, db.FilterEq("starred_by_did", profile.UserDid))

     

       214
       214
       +
       	stars, err := db.GetRepoStars(s.db, 0, db.FilterEq("did", profile.UserDid))

     

       215
       215
        
       	if err != nil {

     

       216
       216
        
       		l.Error("failed to get stars", "err", err)

     

       217
       217
        
       		s.pages.Error500(w)

     
···

       219
       219
        
       	}

     

       220
       220
        
       	var repos []models.Repo

     

       221
       221
        
       	for _, s := range stars {

     

       222
       222
       -
       		if s.Repo != nil {

     

       223
       223
       -
       			repos = append(repos, *s.Repo)

     

       224
       224
       -
       		}

     

       222
       222
       +
       		repos = append(repos, *s.Repo)

     

       225
       223
        
       	}

     

       226
       224
        
       

     

       227
       225
        
       	err = s.pages.ProfileStarred(w, pages.ProfileStarredParams{

+9 -13

appview/state/star.go

···

       57
       57
        
       		log.Println("created atproto record: ", resp.Uri)

     

       58
       58
        
       

     

       59
       59
        
       		star := &models.Star{

     

       60
       60
       -
       			StarredByDid: currentUser.Did,

     

       61
       61
       -
       			RepoAt:       subjectUri,

     

       62
       62
       -
       			Rkey:         rkey,

     

       60
       60
       +
       			Did:    currentUser.Did,

     

       61
       61
       +
       			RepoAt: subjectUri,

     

       62
       62
       +
       			Rkey:   rkey,

     

       63
       63
        
       		}

     

       64
       64
        
       

     

       65
       65
        
       		err = db.AddStar(s.db, star)

     
···

       75
       75
        
       

     

       76
       76
        
       		s.notifier.NewStar(r.Context(), star)

     

       77
       77
        
       

     

       78
       78
       -
       		s.pages.RepoStarFragment(w, pages.RepoStarFragmentParams{

     

       78
       78
       +
       		s.pages.StarBtnFragment(w, pages.StarBtnFragmentParams{

     

       79
       79
        
       			IsStarred: true,

     

       80
       80
       -
       			RepoAt:    subjectUri,

     

       81
       81
       -
       			Stats: models.RepoStats{

     

       82
       82
       -
       				StarCount: starCount,

     

       83
       83
       -
       			},

     

       80
       80
       +
       			SubjectAt: subjectUri,

     

       81
       81
       +
       			StarCount: starCount,

     

       84
       82
        
       		})

     

       85
       83
        
       

     

       86
       84
        
       		return

     
···

       117
       115
        
       

     

       118
       116
        
       		s.notifier.DeleteStar(r.Context(), star)

     

       119
       117
        
       

     

       120
       120
       -
       		s.pages.RepoStarFragment(w, pages.RepoStarFragmentParams{

     

       118
       118
       +
       		s.pages.StarBtnFragment(w, pages.StarBtnFragmentParams{

     

       121
       119
        
       			IsStarred: false,

     

       122
       122
       -
       			RepoAt:    subjectUri,

     

       123
       123
       -
       			Stats: models.RepoStats{

     

       124
       124
       -
       				StarCount: starCount,

     

       125
       125
       -
       			},

     

       120
       120
       +
       			SubjectAt: subjectUri,

     

       121
       121
       +
       			StarCount: starCount,

     

       126
       122
        
       		})

     

       127
       123
        
       

     

       128
       124
        
       		return

+14 -2

appview/strings/strings.go

···

       148
       148
        
       		showRendered = r.URL.Query().Get("code") != "true"

     

       149
       149
        
       	}

     

       150
       150
        
       

     

       151
       151
       +
       	starCount, err := db.GetStarCount(s.Db, string.AtUri())

     

       152
       152
       +
       	if err != nil {

     

       153
       153
       +
       		l.Error("failed to get star count", "err", err)

     

       154
       154
       +
       	}

     

       155
       155
       +
       	user := s.OAuth.GetUser(r)

     

       156
       156
       +
       	isStarred := false

     

       157
       157
       +
       	if user != nil {

     

       158
       158
       +
       		isStarred = db.GetStarStatus(s.Db, user.Did, string.AtUri())

     

       159
       159
       +
       	}

     

       160
       160
       +
       

     

       151
       161
        
       	s.Pages.SingleString(w, pages.SingleStringParams{

     

       152
       152
       -
       		LoggedInUser: s.OAuth.GetUser(r),

     

       162
       162
       +
       		LoggedInUser: user,

     

       153
       163
        
       		RenderToggle: renderToggle,

     

       154
       164
        
       		ShowRendered: showRendered,

     

       155
       155
       -
       		String:       string,

     

       165
       165
       +
       		String:       &string,

     

       156
       166
        
       		Stats:        string.Stats(),

     

       167
       167
       +
       		IsStarred:    isStarred,

     

       168
       168
       +
       		StarCount:    starCount,

     

       157
       169
        
       		Owner:        id,

     

       158
       170
        
       	})

     

       159
       171
        
       }

-25

knotserver/router.go

···

       5
       5
        
       	"fmt"

     

       6
       6
        
       	"log/slog"

     

       7
       7
        
       	"net/http"

     

       8
       8
       -
       	"strings"

     

       9
       8
        
       

     

       10
       9
        
       	"github.com/go-chi/chi/v5"

     

       11
       10
        
       	"tangled.org/core/idresolver"

     
···

       80
       79
        
       	})

     

       81
       80
        
       

     

       82
       81
        
       	r.Route("/{did}", func(r chi.Router) {

     

       83
       83
       -
       		r.Use(h.resolveDidRedirect)

     

       84
       82
        
       		r.Route("/{name}", func(r chi.Router) {

     

       85
       83
        
       			// routes for git operations

     

       86
       84
        
       			r.Get("/info/refs", h.InfoRefs)

     
···

       115
       113
        
       	}

     

       116
       114
        
       

     

       117
       115
        
       	return xrpc.Router()

     

       118
       118
       -
       }

     

       119
       119
       -
       

     

       120
       120
       -
       func (h *Knot) resolveDidRedirect(next http.Handler) http.Handler {

     

       121
       121
       -
       	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {

     

       122
       122
       -
       		didOrHandle := chi.URLParam(r, "did")

     

       123
       123
       -
       		if strings.HasPrefix(didOrHandle, "did:") {

     

       124
       124
       -
       			next.ServeHTTP(w, r)

     

       125
       125
       -
       			return

     

       126
       126
       -
       		}

     

       127
       127
       -
       

     

       128
       128
       -
       		trimmed := strings.TrimPrefix(didOrHandle, "@")

     

       129
       129
       -
       		id, err := h.resolver.ResolveIdent(r.Context(), trimmed)

     

       130
       130
       -
       		if err != nil {

     

       131
       131
       -
       			// invalid did or handle

     

       132
       132
       -
       			h.l.Error("failed to resolve did/handle", "handle", trimmed, "err", err)

     

       133
       133
       -
       			http.Error(w, fmt.Sprintf("failed to resolve did/handle: %s", trimmed), http.StatusInternalServerError)

     

       134
       134
       -
       			return

     

       135
       135
       -
       		}

     

       136
       136
       -
       

     

       137
       137
       -
       		suffix := strings.TrimPrefix(r.URL.Path, "/"+didOrHandle)

     

       138
       138
       -
       		newPath := fmt.Sprintf("/%s/%s?%s", id.DID.String(), suffix, r.URL.RawQuery)

     

       139
       139
       -
       		http.Redirect(w, r, newPath, http.StatusTemporaryRedirect)

     

       140
       140
       -
       	})

     

       141
       116
        
       }

     

       142
       117
        
       

     

       143
       118
        
       func (h *Knot) configureOwner() error {

Compare changes